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

Bump media-sound/beets to 1.4.9 #12582

Closed
wants to merge 2 commits into from
Closed

Bump media-sound/beets to 1.4.9 #12582

wants to merge 2 commits into from

Conversation

lovesegfault
Copy link
Contributor

No description provided.

@gentoo-bot
Copy link

Copyright policy change

Please note that on 2018-09-15 Trustees have approved new Gentoo copyright policy. All contributions made to Gentoo need to follow this policy. If you include the Signed-off-by line in your commit message, you indicate that you have read the policy and agree to its terms. For more detailed explanation, please see the new Gentoo copyright policy explained article.

Pull Request assignment

Submitter: @lovesegfault
Areas affected: ebuilds
Packages affected: media-sound/beets

media-sound/beets: @gentoo/sound

Linked bugs

Bugs linked: 689182


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

Docs: Code of ConductCopyright policy (expl.) ● DevmanualGitHub PRsProxy-maint guide

@gentoo-bot gentoo-bot added assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR. labels Jul 30, 2019
@lovesegfault
Copy link
Contributor Author

Rebased

media-sound/beets/beets-1.4.9.ebuild Show resolved Hide resolved
media-sound/beets/beets-1.4.9.ebuild Outdated Show resolved Hide resolved
media-sound/beets/beets-1.4.9.ebuild Outdated Show resolved Hide resolved
media-sound/beets/beets-1.4.9.ebuild Outdated Show resolved Hide resolved
media-sound/beets/beets-1.4.9.ebuild Outdated Show resolved Hide resolved
dev-python/setuptools[${PYTHON_USEDEP}]
"

RESTRICT="test" # tests broken in 1.4.3 already
Copy link
Member

Choose a reason for hiding this comment

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

Have they been fixed for 1.4.9 though? ;)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, but they require a large refactoring of the ebuild which will come in a revbump after I merge some more packages.

media-sound/beets/beets-1.4.9.ebuild Outdated Show resolved Hide resolved
done

if ! use mpd; then
rm -f test/test_player.py || die
Copy link
Member

Choose a reason for hiding this comment

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

Ǹo -f please, i makes it so rm never dies. I'm not a huge fan of this style of removing plugins, since switching a USE flag requires re-compiling this again, while if you just left everything in place, would the functionality be available after installing said package?
Well since this looks like a small package I don't mind so much about recompiling it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I completely agree with you, I am working towards a complete rewrite of this ebuild that removes this completely. I'll be submitting new pkg requests in the next couple of days. For now I'm leaving it untouched in favour of a simpler bump.

media-sound/beets/beets-1.4.9.ebuild Outdated Show resolved Hide resolved
lovesegfault and others added 2 commits August 29, 2019 22:30
Package-Manager: Portage-2.3.73, Repoman-2.3.17
Signed-off-by: Bernardo Meurer <bernardo@standard.ai>
Closes: https://bugs.gentoo.org/689182
Package-Manager: Portage-2.3.69, Repoman-2.3.16
Signed-off-by: Bernardo Meurer <bernardo@standard.ai>
@lovesegfault
Copy link
Contributor Author

@juippis Pushed the changes.

This is an admittedly poor bump, but to make this ebuild good will take a lot of work, I'm 60% of the way there. We can either:

  1. Merge this, and then merge the re-write as a revbump later
  2. Remove the changes here, merge my addition as maintainer, merge re-write as 1.4.9 later

I've been running 1.4.9 off of this ebuild for almost a month now with no problems, FWIW.

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2019-08-30 06:02 UTC
Newest commit scanned: ec67b71
Status: ✅ good

Issues already there before the PR (double-check them):
https://qa-reports.gentoo.org/output/gentoo-ci/aa2b5ea/output.html#sci-libs/ViSP

discogs? ( dev-python/discogs-client[${PYTHON_USEDEP}] )
ffmpeg? ( media-video/ffmpeg:0[encode] )
gstreamer? (
media-libs/gst-plugins-good:1.0
Copy link
Member

Choose a reason for hiding this comment

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

Remember to sort this properly on the revbump.

media-libs/gst-plugins-good:1.0
media-libs/gst-plugins-bad:1.0
)
icu? ( dev-db/sqlite[icu] )
Copy link
Member

Choose a reason for hiding this comment

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

This is slotted if I recall correctly.

@lovesegfault lovesegfault deleted the beets-1.4.9 branch August 30, 2019 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR.
Projects
None yet
4 participants