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-sound/qtractor: Bump to 0.9.4 #11011

Closed

Conversation

simonvanderveldt
Copy link
Contributor

Qt4 support was dropped in it's entirety so I've removed the econf flag for it.
With the removal of Qt4 support the --with-qt5 flag was also dropped, so removed that as well. Emering/compilation still works fine.
Also validated that running and opening/using projects works correctly.

P.S. This is a bump and drop in 1 commit, I'm not sure if that's OK or not. If not just let me know I'll split it into 2 commits.

@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: @simonvanderveldt
Areas affected: ebuilds
Packages affected: media-sound/qtractor

media-sound/qtractor: [proaudio (project)]

At least one of the listed packages is maintained entirely by non-GitHub developers!

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.

If you do not receive any reply to this pull request, please open or link a bug to attract the attention of maintainers.

Missing GCO sign-off

Please read the terms of Gentoo Certificate of Origin and acknowledge them by adding a sign-off to all your commits.


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 need assignment It was impossible to assign the PR correctly. Please assign it manually. no bug found No Bug/Closes found in the commits. no signoff One or more commits do not indicate GCO sign-off. labels Feb 9, 2019
Copy link
Member

@a17r a17r left a comment

Choose a reason for hiding this comment

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

P.S. This is a bump and drop in 1 commit, I'm not sure if that's OK or not. If not just let me know I'll split it into 2 commits.

Indeed, please keep them separate commits. Also, only drop it if you are absolutely sure there is no regression over 0.9.2 that may be worth keeping the old version for some time. ;)

Also, for EAPI-7 you should move all content of DEPEND (except RDEPEND inheritance) into BDEPEND instead.

@a17r
Copy link
Member

a17r commented Feb 9, 2019

Also, please add your signoff so we can merge your PR.

Signed-off-by: Simon van der Veldt <simon.vanderveldt@gmail.com>
Signed-off-by: Simon van der Veldt <simon.vanderveldt@gmail.com>
@simonvanderveldt
Copy link
Contributor Author

Indeed, please keep them separate commits. Also, only drop it if you are absolutely sure there is no regression over 0.9.2 that may be worth keeping the old version for some time. ;)

Done. There are no real significant changes between 0.9.2 and 0.9.4 and it's all working fine so I think this is fine. Rui only does small changes nowadays so breakage seldom happens :)

Also, for EAPI-7 you should move all content of DEPEND (except RDEPEND inheritance) into BDEPEND instead.

I've incorporated this change, also moved DEPEND into RDEPEND instead of the other way around since that seems to be the suggested way of doing this. Hope this is correct.

Added my signoff to the commits as well.

Copy link
Member

@a17r a17r left a comment

Choose a reason for hiding this comment

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

 * One or more compressed files were found in docompress-ed directories.
 * Please fix the ebuild not to install compressed files (manpages,
 * documentation) when automatic compression is used:
 * 
 *   /usr/share/man/man1/qtractor.fr.1.gz
 *   /usr/share/man/man1/qtractor.1.gz

Please look into fixing those (likely needs buildsys patching).

@simonvanderveldt
Copy link
Contributor Author

simonvanderveldt commented Feb 10, 2019

Yeah, I noticed those. That was already the case for the old(er) version(s) as well though, didn't want to add changes unrelated to the bump.

What would be the suggested way to fix this?
Add a patch that removes this + the next line https://github.com/rncbc/qtractor/blob/master/Makefile.in#L348
and then add a doman for the man pages?

@a17r
Copy link
Member

a17r commented Feb 10, 2019

Removing line 349 should be enough, you can add the fix in a separate commit ofc.

@simonvanderveldt
Copy link
Contributor Author

@a17r I've added a patch that should address the QA issue.

Signed-off-by: Simon van der Veldt <simon.vanderveldt@gmail.com>
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2019-02-10 23:41 UTC
Newest commit scanned: 2ffcb5a
Status: ✅ good

No issues found

@simonvanderveldt simonvanderveldt deleted the qtractor-bump-0.9.4 branch February 14, 2019 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
need assignment It was impossible to assign the PR correctly. Please assign it manually. no bug found No Bug/Closes found in the commits. no signoff One or more commits do not indicate GCO sign-off.
Projects
None yet
4 participants