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

typo in packaging configuration fixed, now builds packages correctly #696

Merged
merged 1 commit into from
Jul 10, 2023

Conversation

beadon
Copy link
Contributor

@beadon beadon commented Jul 10, 2023

Looks like a few letters were jumbled for some reason. This broke cpack.

Works now!

@EionRobb EionRobb merged commit 5daf79d into EionRobb:master Jul 10, 2023
@EionRobb
Copy link
Owner

Thanks! 🙂

@vitalyster
Copy link
Contributor

This breaks output filename at least for DEB files

@beadon
Copy link
Contributor Author

beadon commented Jul 10, 2023

This breaks output filename at least for DEB files

Yes, I noticed that the DEB files are not named properly, but cpack wasn't working AT ALL before this. So I chose to not deep-dive into the reason why the filename was not right.

If you like I can fix the package name too ? It's beyond the scope of just fixing the typo.

@vitalyster
Copy link
Contributor

If the only issue was "generator not specified" you can provide generator on command line, like this cpack -G DEB
That "typo" line should be removed at all

@beadon
Copy link
Contributor Author

beadon commented Jul 10, 2023

If the only issue was "generator not specified" you can provide generator on command line, like this cpack -G DEB That "typo" line should be removed at all

no, your guess was not correct.

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.

3 participants