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

feat: add alternative headless and libadwaita builds #227

Merged
merged 14 commits into from
Dec 9, 2023

Conversation

ilya-zlobintsev
Copy link
Owner

No description provided.

Comment on lines 23 to 25
- name: Set libadwaita target
if: contains(matrix.target-os, 'libadwaita')
run: sed -i 's/\&\& make/\&\& make build-release-libadwaita/g' pkg/recipes/lact/recipe.yml
Copy link
Contributor

Choose a reason for hiding this comment

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

woulnd't it be better to keep the original recipe intact and just create another one for libadwaita?

Copy link
Owner Author

Choose a reason for hiding this comment

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

I'm trying to avoid the duplicate build steps between the recipes, ideally this would be done by passing some build flag when building the package, but unfortunately it doesn't seem possible

Copy link
Owner Author

@ilya-zlobintsev ilya-zlobintsev Dec 9, 2023

Choose a reason for hiding this comment

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

I've ended up creating separate recipes after all so it's easier to manage

@ilya-zlobintsev ilya-zlobintsev changed the title feat: add alternative builds with libadwaita feat: add alternative headless and libadwaita builds Dec 9, 2023
@ilya-zlobintsev ilya-zlobintsev marked this pull request as ready for review December 9, 2023 14:56
@GabMus
Copy link
Contributor

GabMus commented Dec 9, 2023

LGTM :)

@ilya-zlobintsev ilya-zlobintsev merged commit 999418a into master Dec 9, 2023
15 checks passed
@ilya-zlobintsev ilya-zlobintsev deleted the alternative-libadwaita branch December 9, 2023 19:13
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.

2 participants