Skip to content

Users/astarche/cmako next generation#6

Open
astarche wants to merge 13 commits intomainfrom
users/astarche/cmako-next-generation
Open

Users/astarche/cmako next generation#6
astarche wants to merge 13 commits intomainfrom
users/astarche/cmako-next-generation

Conversation

@astarche
Copy link
Owner

@astarche astarche commented Sep 3, 2021

Test

@astarche astarche force-pushed the users/astarche/cmako-next-generation branch from bba6131 to d14d16f Compare September 9, 2021 13:19
from attributes import attributes
from enums import enums
from config import config
from .functions import functions
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note: the original implementation of this is sort of a hack. References within a package either need to include the package name (generally recommended) or be explicitly relative (like this, because it allows all of these init.pys to be identical). The previous implementation "worked" by munging sys.path and doesn't work if you try to do it twice in one process (because the first global "functions" wins).

@@ -1,4 +1,4 @@
#include <nidcpower/nidcpower_service.h>
#include <nidcpower_service.h>
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The custom cpps are inside the driver library and now just reference by filename (or not at all if they're empty)

with:
access_token: ${{ github.token }}

- name: Set Swap Space
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The nilrt build has been borderline out-of-memory since I added the -j2 flag. This is a way to get around that. TODO: change the @ to a commit SHA (or do something else).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant