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.

Sign up for GitHub

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

Jump to bottom

sys-fs/bees: Updates and cleanups #21320

Closed
wants to merge 6 commits into from
Closed

sys-fs/bees: Updates and cleanups #21320

wants to merge 6 commits into from

Conversation

kakra
Copy link
Contributor

@kakra kakra commented Jun 19, 2021

No description provided.

@gentoo-bot
Copy link

Pull Request assignment

Submitter: @kakra
Areas affected: ebuilds
Packages affected: sys-fs/bees

sys-fs/bees: @kakra, @gentoo/proxy-maint

Linked bugs

No bugs to link found. If your pull request references any of the Gentoo bug reports, please add appropriate GLEP 66 tags to the commit message and request reassignment.

In order to force reassignment and/or bug reference scan, please append [please reassign] to the pull request title.

Docs: Code of Conduct ● Copyright policy (expl.) ● Devmanual ● GitHub PRs ● Proxy-maint guide

@gentoo-bot gentoo-bot added self-maintained The PR changes only packages that are maintained by the submitter (i.e. no need to ask anybody else) assigned PR successfully assigned to the package maintainer(s). labels Jun 19, 2021
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2021-06-19 08:59 UTC
Newest commit scanned: 618346f
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/bf54c32dae/output.html

ionenwks
ionenwks reviewed Jul 7, 2021
View reviewed changes
Copy link
Contributor

@ionenwks ionenwks left a comment

Choose a reason for hiding this comment

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

Sorry for the wait, just a few trivialities that'd be nice to get in.

sys-fs/bees/bees-0.6.5.ebuild Show resolved Hide resolved
sys-fs/bees/bees-0.6.5.ebuild Outdated Show resolved Hide resolved
sys-fs/bees/bees-0.6.5.ebuild Outdated Show resolved Hide resolved
sys-fs/bees/bees-0.6.5.ebuild Show resolved Hide resolved
kakra added 6 commits July 7, 2021 19:30
@kakra
sys-fs/bees-9999: Drop conflicting musl hack
b02b8c9 
An upcoming musl update will add the missing function.

Package-Manager: Portage-3.0.20, Repoman-3.0.2
Signed-off-by: Kai Krakow <kai@kaishome.de>
@kakra
sys-fs/bees-9999: Fix installation with eprefix
b085a57 
Package-Manager: Portage-3.0.20, Repoman-3.0.2
Signed-off-by: Kai Krakow <kai@kaishome.de>
@kakra
sys-fs/bees: Bump to v0.6.5
76d89f7 
Fixes: https://bugs.gentoo.org/753542
Fixes: https://bugs.gentoo.org/725996
Package-Manager: Portage-3.0.20, Repoman-3.0.2
Signed-off-by: Kai Krakow <kai@kaishome.de>
@kakra
sys-fs/bees: Drop old
3f24dd5 
Package-Manager: Portage-3.0.20, Repoman-3.0.2
Signed-off-by: Kai Krakow <kai@kaishome.de>
@kakra
sys-fs/bees-9999: Fix style
59e1453 
Package-Manager: Portage-3.0.20, Repoman-3.0.2
Signed-off-by: Kai Krakow <kai@kaishome.de>
@kakra
sys-fs/bees: Inherit toolchain-funcs
4cf12da 
Package-Manager: Portage-3.0.20, Repoman-3.0.2
Signed-off-by: Kai Krakow <kai@kaishome.de>
kakra
kakra commented Jul 7, 2021
View reviewed changes
Copy link
Contributor Author

@kakra kakra left a comment

Choose a reason for hiding this comment

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

@ionenwks All review comments should be resolved.

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2021-07-07 17:44 UTC
Newest commit scanned: 4cf12da
Status: ❌ broken

New issues caused by PR:
https://qa-reports.gentoo.org/output/gentoo-ci/bb2d19e1f4/output.html#sys-fs/bees

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/bb2d19e1f4/output.html

ionenwks
ionenwks approved these changes Jul 7, 2021
View reviewed changes
Copy link
Contributor

@ionenwks ionenwks left a comment

Choose a reason for hiding this comment

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

Most commands already know about EPREFIX, it's when specifying paths manually that it's needed -- so that last one wasn't needed :)

Don't worry about it, trivial so I'll fix on merge and squash commits a bit if you don't mind.

@kakra
Copy link
Contributor Author

kakra commented Jul 7, 2021

Most commands already know about EPREFIX, it's when specifying paths manually that it's needed -- so that last one wasn't needed :)

Oh, you mean the systemd thingy?

Don't worry about it, trivial so I'll fix on merge and squash commits a bit if you don't mind.

Thanks. Please do if it means no efforts.

@kakra
Copy link
Contributor Author

kakra commented Jul 7, 2021

BTW: repoman didn't complain about it, but I see the CI bot did.

@ionenwks
Copy link
Contributor

ionenwks commented Jul 7, 2021

BTW: repoman didn't complain about it, but I see the CI bot did.

dev-util/pkgcheck is a bit smarter than repoman, it's what CI uses

Also, 0.6.5 seems to have fixed the clang bug on b.g.o. I can reproduce with 0.6.4 but not 0.6.5, so I'll add it to the list.

@gentoo-bot gentoo-bot closed this in 6ad009c Jul 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ionenwks ionenwks ionenwks approved these changes

Assignees
No one assigned
Labels
assigned PR successfully assigned to the package maintainer(s). self-maintained The PR changes only packages that are maintained by the submitter (i.e. no need to ask anybody else)
Projects
None yet
Milestone
No milestone
4 participants
@kakra @gentoo-bot @gentoo-repo-qa-bot @ionenwks