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

Generate output into corresponding library directory #103

Merged
merged 1 commit into from
Aug 15, 2023

Conversation

ubruhin
Copy link
Member

@ubruhin ubruhin commented Aug 9, 2023

Instead of generating the output into a directory like "out/led/pkg/", use the actual library directory where the files need to be added to, for example "out/LibrePCB_Base.lplib/pkg". This makes it much easier to copy the generated elements into the libraries.

In addition, it is less error-prone since the library assignment is now stored in the generators (until now it was cumbersome to find out which files need to be copied into which library - especially if a single script generates output for multiple libraries).

I've run all generators to verify that their new output directory is correct.

Instead of generating the output into a directory like "out/led/pkg/",
use the actual library directory where the files need to be added to,
for example "out/LibrePCB_Base.lplib/pkg". This makes it much easier
to copy the generated elements into the libraries. In addition, it
is less error-prone since the library assignment is now stored in the
generators (currently it's cumbersome to find out which files need to
be copied into which library).
@ubruhin ubruhin added the enhancement New feature or request label Aug 9, 2023
@ubruhin ubruhin requested review from dbrgn and rnestler August 9, 2023 13:03
@ubruhin
Copy link
Member Author

ubruhin commented Aug 14, 2023

Ping @dbrgn @rnestler 😉

Copy link
Member

@dbrgn dbrgn left a comment

Choose a reason for hiding this comment

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

I didn't review every line, but looks good in general!

@ubruhin
Copy link
Member Author

ubruhin commented Aug 15, 2023

Thanks @dbrgn! 👍

@ubruhin ubruhin merged commit 241a72a into master Aug 15, 2023
6 checks passed
@ubruhin ubruhin deleted the output-directories-by-library branch August 15, 2023 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

Successfully merging this pull request may close these issues.

2 participants