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

media-plugins/alsaequal: Fix https://bugs.gentoo.org/499298 reported by Tiago Resende #6255

Closed
wants to merge 3 commits into from

Conversation

bassdr
Copy link

@bassdr bassdr commented Nov 21, 2017

Bug: https://bugs.gentoo.org/499298

Package-Manager: Portage-2.3.13, Repoman-2.3.3

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull Request assignment

Areas affected: ebuilds
Packages affected: media-plugins/alsaequal

media-plugins/alsaequal: @gentoo/proaudio

No bug reference found in commit messages. If you would like to reference bugs in the pull request, please make sure to use GLEP 66 tags in the commit message.

If there is no bug open for this pull request, please open one. Otherwise, please link it to the commit messages as noted above.

@gentoo-repo-qa-bot gentoo-repo-qa-bot added assigned PR successfully assigned to the package maintainer(s). no bug found No Bug/Closes found in the commits. labels Nov 21, 2017
@bassdr bassdr closed this Nov 21, 2017
@bassdr bassdr reopened this Nov 21, 2017
@gentoo-repo-qa-bot
Copy link
Collaborator

👍 All QA issues have been fixed!

@FuzzyGophers
Copy link
Contributor

@bassdr Do you still intend to work on this PR?

@bassdr
Copy link
Author

bassdr commented Apr 5, 2018

Yes, why not? I did not have any feedback though... I am using alsaequal on one of my box and it works using my fork...

@@ -1,16 +1,16 @@
# Copyright 1999-2014 Gentoo Foundation
# Copyright 1999-2017 Gentoo Foundation
Copy link
Contributor

Choose a reason for hiding this comment

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

Update the copyright

Copy link
Contributor

@FuzzyGophers FuzzyGophers left a comment

Choose a reason for hiding this comment

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

You are performing a rename in Git for this. By doing so, you are dropping the 0.6 ebuild which is stable. This should just be a new ebuild in the tree for 0.7.

Also, adjust the commit to have a proper GLEP66 tag.

e.g. drop "Fix https://bugs.gentoo.org/499298" from the summary and put it in the proper GLEP66 tag.

HOMEPAGE="http://www.thedigitalmachine.net/alsaequal.html"
SRC_URI="http://www.thedigitalmachine.net/tools/${P}.tar.bz2"
DESCRIPTION="A real-time adjustable equalizer plugin for ALSA"
HOMEPAGE="https://github.com/bassdr/alsaequal"
Copy link
Contributor

Choose a reason for hiding this comment

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

Why not use ${PN} here as well?

Copy link
Contributor

@FuzzyGophers FuzzyGophers left a comment

Choose a reason for hiding this comment

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

The patches should remain in the tree as well once you have fixed the ebuild renaming issue.

@FuzzyGophers
Copy link
Contributor

No activity in 45+ days.

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). no bug found No Bug/Closes found in the commits.
Projects
None yet
3 participants