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

games-emulation/ppsspp: bug fix #572606 #740

Closed
wants to merge 201 commits into from
Closed

games-emulation/ppsspp: bug fix #572606 #740

wants to merge 201 commits into from

Conversation

RobinDX
Copy link
Contributor

@RobinDX RobinDX commented Jan 29, 2016

@gentoo-repo-qa-bot
Copy link
Collaborator

😞 The QA check for this pull request has found the following issues:

Issues persisted from underlying repository state:
https://qa-reports.gentoo.org/output/gentoo-ci/a205130/output.html#dev-go/godep
https://qa-reports.gentoo.org/output/gentoo-ci/a205130/output.html#media-video/nvidia-settings

@RobinDX
Copy link
Contributor Author

RobinDX commented Jan 29, 2016

@monsieurp
What happened to QA ?

@monsieurp
Copy link
Member

@RobinDX: I told you a zillion times that you only need to push your changes onto the branch you've created when sending the PR for the bot to pick them up again and rerun the QA tests; in the case of this PR: RobinDX:psp. There's no need to create a PR every time you want the QA tests to run. Thank you.

@monsieurp
Copy link
Member

Since this fix modifies the install path of the ppsspp binary, we have to create a new revision. Please, undo your changes in both ebuilds and create two new revisions, namely ppsspp-1.1.0-r1.ebuild and ppsspp-1.1.1-r1.ebuild with your fix in it. Cheers.

@monsieurp monsieurp added bugfix assigned PR successfully assigned to the package maintainer(s). labels Jan 29, 2016
@monsieurp
Copy link
Member

and DO NOT close this PR and send a new one! 😠

@RobinDX
Copy link
Contributor Author

RobinDX commented Jan 30, 2016

@monsieurp
Jobs done

@mgorny
Copy link
Member

mgorny commented Jan 30, 2016

@RobinDX, please rebase your commits against fresh repository. This pull request is based on pre-GLEP67 ::gentoo repository which triggers so many QA issues that the script fails with lack of resources to process them all. Otherwise, it would throw around 18k errors here ;-P.

nigoro and others added 21 commits January 31, 2016 00:46
…ver into proper directory (bug #534152)

Package-Manager: portage-2.2.26
RepoMan-Options: --ignore-arches
… in the .ebuild

Package-Manager: portage-2.2.26
RepoMan-Options: --ignore-arches
Package-Manager: portage-2.2.26
…(bug #572730 by Martijn Schmidt).

Package-Manager: portage-2.2.27
Package-Manager: portage-2.2.27
Package-Manager: portage-2.2.27
…(bug #572730 by Martijn Schmidt).

Package-Manager: portage-2.2.27
Package-Manager: portage-2.2.27
…(bug #572730 by Martijn Schmidt).

Package-Manager: portage-2.2.27
Package-Manager: portage-2.2.26
Package-Manager: portage-2.2.26
…72854).

Package-Manager: portage-2.2.27
Signed-off-by: Lars Wendler <polynomial-c@gentoo.org>
Package-Manager: portage-2.2.27
Signed-off-by: Lars Wendler <polynomial-c@gentoo.org>
Do not define SRC_URI and S (which was the default).
Convert latex-package_has_tetex_3() to a dummy function for backwards
compatibility and remove its documentation.
In EAPI 6, don't inherit base.eclass and don't define
latex-package_has_tetex_3().
Package-Manager: portage-2.2.26
…0} - remove old.

Package-Manager: portage-2.2.26
Signed-off-by: Agostino Sarubbo <ago@gentoo.org>
jlec and others added 20 commits January 31, 2016 00:47
Package-Manager: portage-2.2.27
Signed-off-by: Justin Lecher <jlec@gentoo.org>
Package-Manager: portage-2.2.27
Signed-off-by: Justin Lecher <jlec@gentoo.org>
Gentoo-Bug: https://bugs.gentoo.org/show_bug.cgi?id=573372

Package-Manager: portage-2.2.27
Signed-off-by: Justin Lecher <jlec@gentoo.org>
Package-Manager: portage-2.2.26
revbump uses new SRC_URI fetching an adjusted tarball with a pre-built
set of docs; add IUSE doc and doc install in the usual form
Also bump to EAPI 6, patch submitted by E. Myers via the gentoo bug

Gentoo bug: #573316

Package-Manager: portage-2.2.26
Package-Manager: portage-2.2.27
Signed-off-by: Justin Lecher <jlec@gentoo.org>
case statement ignored EAPI > 5

Gentoo-Bugs: https://bugs.gentoo.org/show_bug.cgi?id=573368

Signed-off-by: Justin Lecher <jlec@gentoo.org>
Package-Manager: portage-2.2.26
Package-Manager: portage-2.2.26
Package-Manager: portage-2.2.26
Package-Manager: portage-2.2.26
Package-Manager: portage-2.2.26
Package-Manager: portage-2.2.26
Package-Manager: portage-2.2.27
Signed-off-by: Alexis Ballier <aballier@gentoo.org>
Package-Manager: portage-2.2.27
Signed-off-by: Alexis Ballier <aballier@gentoo.org>
Package-Manager: portage-2.2.27
Signed-off-by: Alexis Ballier <aballier@gentoo.org>
Package-Manager: portage-2.2.27
@RobinDX
Copy link
Contributor Author

RobinDX commented Jan 30, 2016

@mgorny
So what should I do ?

@RobinDX RobinDX closed this Jan 30, 2016
@RobinDX RobinDX deleted the psp branch January 30, 2016 17:22
@RobinDX RobinDX restored the psp branch January 30, 2016 17:23
@RobinDX RobinDX deleted the psp branch January 30, 2016 17:24
@RobinDX RobinDX restored the psp branch January 30, 2016 17:25
@RobinDX RobinDX deleted the psp branch January 30, 2016 17:26
@RobinDX
Copy link
Contributor Author

RobinDX commented Jan 30, 2016

I think I have do something wrong with git rebase.

@gentoo-repo-qa-bot
Copy link
Collaborator

😞 The QA check for this pull request has found the following issues:

New issues:
https://qa-reports.gentoo.org/output/gentoo-ci/a8fa0f7/output.html#sys-cluster/nova

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).
Projects
None yet