-
Notifications
You must be signed in to change notification settings - Fork 65
Just extraction #818
base: master
Are you sure you want to change the base?
Just extraction #818
Conversation
1. use common variable for module & target caches 2. change cache directories if exporting 3. export entire built directory to destination 4. missing relative 'include_directories' fixed
e.g. simplelog wayyyy too much copypaste
there are zero references to the project directory worth trying on ci
This reverts commit afc8202.
docs/tutorial/yotta_link.md
Outdated
| [1/3] Building C object ym/simplelog/source/CMakeFiles/simplelog.dir/path/to/helloyotta/yotta_modules/simplelog/source/simplelog.c.o | ||
| FAILED: /usr/bin/cc -I/path/to/helloyotta -I/path/to/helloyotta/yotta_modules/simplelog -I/path/to/helloyotta/yotta_modules/simplelog/source -O2 -g -DNDEBUG -include "/path/to/helloyotta/build/x86-osx-native/yotta_config.h" -MMD -MT ym/simplelog/source/CMakeFiles/simplelog.dir/path/to/helloyotta/yotta_modules/simplelog/source/simplelog.c.o -MF ym/simplelog/source/CMakeFiles/simplelog.dir/path/to/helloyotta/yotta_modules/simplelog/source/simplelog.c.o.d -o ym/simplelog/source/CMakeFiles/simplelog.dir/path/to/helloyotta/yotta_modules/simplelog/source/simplelog.c.o -c /path/to/helloyotta/yotta_modules/simplelog/source/simplelog.c | ||
| [1/3] Building C object modules/simplelog/source/CMakeFiles/simplelog.dir/path/to/helloyotta/yotta_modules/simplelog/source/simplelog.c.o | ||
| FAILED: /usr/bin/cc -I/path/to/helloyotta -I/path/to/helloyotta/yotta_modules/simplelog -I/path/to/helloyotta/yotta_modules/simplelog/source -O2 -g -DNDEBUG -include "/path/to/helloyotta/build/x86-osx-native/yotta_config.h" -MMD -MT modules/simplelog/source/CMakeFiles/simplelog.dir/path/to/helloyotta/yotta_modules/simplelog/source/simplelog.c.o -MF modules/simplelog/source/CMakeFiles/simplelog.dir/path/to/helloyotta/yotta_modules/simplelog/source/simplelog.c.o.d -o modules/simplelog/source/CMakeFiles/simplelog.dir/path/to/helloyotta/yotta_modules/simplelog/source/simplelog.c.o -c /path/to/helloyotta/yotta_modules/simplelog/source/simplelog.c |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should still say ym not modules
docs/tutorial/yotta_link.md
Outdated
|
|
||
| ``` | ||
| [1/3] Building C object ym/simplelog/source/CMakeFiles/simplelog.dir/path/to/helloyotta/yotta_modules/simplelog/source/simplelog.c.o | ||
| [1/3] Building C object modules/simplelog/source/CMakeFiles/simplelog.dir/path/to/helloyotta/yotta_modules/simplelog/source/simplelog.c.o |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ym
| by a dependency name (if there were no `ym/` at the start of the path then the | ||
| by a dependency name (if there were no `modules/` at the start of the path then the | ||
| object would belong to the top-level module/app being built). So from | ||
| `/ym/simplelog/...` we can tell that the failed file is in the `simplelog` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ym
yotta/build.py
Outdated
| parser.add_argument('-x', '--export', dest='export', | ||
| default=False, const=True, nargs='?', | ||
| help=( | ||
| 'Export mode. If flag is set, generates builds without reference to Yotta.' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yotta not Yotta, or perhaps remove this comment? e.g. Export mode. If flag is set, generates a buildable folder structure in the specified folder.
yotta/test_subcommand.py
Outdated
| submod = False | ||
| module = toplevel_module | ||
| # <topdir> /ym/<submod>/ym/<submod2>/somedir/somedir --> submod2 | ||
| # <topdir> /modules/<submod>/modules/<submod2>/somedir/somedir --> submod2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ym ??
Add changes from @sami Haapakoski
Adds a
'-x', '--export'option to YottaSalient points:
self.relative)