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

apparmor: fix packaging #15875

Merged
merged 1 commit into from
Jun 17, 2021
Merged

apparmor: fix packaging #15875

merged 1 commit into from
Jun 17, 2021

Conversation

CodeFetch
Copy link
Contributor

Removes the BUILDONLY flag as otherwise no packages will be created.

@oskarirauta

@neheb
Copy link
Contributor

neheb commented Jun 16, 2021

Commit sha: 6e5880b, Author: Vincent Wiemann, Committer: GitHub; Expected "Vincent Wiemann [email protected]", but got "Vincent Wiemann [email protected]".

Removes the BUILDONLY flag as otherwise no packages will be created.

Signed-off-by: Vincent Wiemann <[email protected]>
@CodeFetch
Copy link
Contributor Author

@neheb Thanks for the hint! One more reason to not use Github's web interface for committing.

@neheb neheb merged commit 89e9a1a into openwrt:master Jun 17, 2021
@BKPepe
Copy link
Member

BKPepe commented Jun 17, 2021

Where is approve from maintaner? 👀

@CodeFetch
Copy link
Contributor Author

@BKPepe This was a point of discussion right from the beginning. #15643 (comment)

We have tested it and no packages will be created with the BUILDONLY flag set and the files are missing in rootfs. @oskarirauta said that something did not work, but I can't confirm that. The BUILDONLY flag is used for packages which are only used as build dependencies. It's wrong to use it here.

@BKPepe
Copy link
Member

BKPepe commented Jun 17, 2021

Well, don't take it wrong, but what's the point of being maintainer if he didn't approve it and it is merged no matter what? 1 or 2 days from my point of view are not enough to review the things!

Based on your comment, I could see that we don't need to have maintainers at all and that's why they are leaving like #15853

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