Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

File organization #3

Open
lhstrh opened this issue Oct 16, 2021 · 3 comments
Open

File organization #3

lhstrh opened this issue Oct 16, 2021 · 3 comments
Assignees
Labels
enhancement Enhancement of existing feature

Comments

@lhstrh
Copy link
Member

lhstrh commented Oct 16, 2021

As suggested by @Soroosh129, the ctarget.h file was put in include/ and ctarget.c was put in lib/. For the sake of consistency, we should probably split the files currently in core/ between include/core/ and lib/core/.

@Soroosh129 Soroosh129 removed their assignment Oct 16, 2021
@Soroosh129
Copy link
Contributor

I think core/include and core/lib might be better.

@lhstrh
Copy link
Member Author

lhstrh commented Oct 16, 2021

Your call entirely.

@lhstrh lhstrh added the enhancement Enhancement of existing feature label Oct 20, 2021
@lhstrh
Copy link
Member Author

lhstrh commented Oct 20, 2021

Following up on our conversation, here's a draft proposal for how to do this. Addressing this issue would be necessary ahead of #8 and might be a bit tricky. There is no rush with this effort, but let's make sure that we have a solid plan for doing this.
Here's what I gather so far. Feel free to go in and edit this:

core/include/platform.h
core/include/pqueue.h
...
core/lib/pqueue.c
core/lib/tag.c
...
core/platform/include/lf_C11_threads_support.h
...
federated/include/federate.h -> xronos/include/federate.h (i.e., later on move to separate xronos repo)
federated/lib/federate.c -> xronos/lib/federate.c
...
federated/include/rti.h -> xronos/include/rti.h
federated/lib/rti.c -> xronos/lib/rti.c
...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement of existing feature
Projects
None yet
Development

No branches or pull requests

2 participants