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

Add udpated ebuild so that the package supports current gcc #15903

Closed
wants to merge 9 commits into from
Closed

Add udpated ebuild so that the package supports current gcc #15903

wants to merge 9 commits into from

Conversation

dr-diem
Copy link
Contributor

@dr-diem dr-diem commented May 21, 2020

Also fixes bugs 669212 & 703090.

@gentoo-bot
Copy link

Pull Request assignment

Submitter: @dr-diem
Areas affected: ebuilds
Packages affected: games-action/dxx-rebirth

games-action/dxx-rebirth: @gentoo/games, totktonada.ru[at]gmail.com

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 assigned PR successfully assigned to the package maintainer(s). no bug found No Bug/Closes found in the commits. no signoff One or more commits do not indicate GCO sign-off. labels May 21, 2020
…nt packages from the upstream project

Signed-off-by: Ian Silvester iansilvester@fastmail.fm
Closes: https://bugs.gentoo.org/669212
Closes: https://bugs.gentoo.org/703090
Though this older ebuild only compiles with gcc 6.5.0 and is therefore outdated, it is not for me to delete it outright

Signed-off-by: Ian Silvester iansilvester@fastmail.fm
…pyright statement to ebuild

Signed-off-by: Ian Silvester iansilvester@fastmail.fm
Signed-off-by: Ian Silvester iansilvester@fastmail.fm
Signed-off-by: Ian Silvester iansilvester@fastmail.fm
…metadata.xml

Signed-off-by: Ian Silvester iansilvester@fastmail.fm
…zip for source code

Signed-off-by: Ian Silvester iansilvester@fastmail.fm
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2020-05-22 15:26 UTC
Newest commit scanned: 95dc9eb
Status: ❌ broken

New issues caused by PR:
https://qa-reports.gentoo.org/output/gentoo-ci/5e647f28ad/output.html#games-action/descent1-freedata
https://qa-reports.gentoo.org/output/gentoo-ci/5e647f28ad/output.html#games-action/descent2-freedata
https://qa-reports.gentoo.org/output/gentoo-ci/5e647f28ad/output.html#games-action/dxx-rebirth

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/5e647f28ad/output.html

@dr-diem dr-diem changed the title Update ebuild to support current gcc Add udpated ebuild so that the package supports current gcc May 24, 2020
@dr-diem
Copy link
Contributor Author

dr-diem commented May 24, 2020

@chewi chewi self-assigned this May 28, 2020
Copy link
Member

@chewi chewi left a comment

Choose a reason for hiding this comment

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

I'm aware you're still working on this but your first two commits are missing sign-off headers so I can't merge this as is. You'll need to rebase this anyway. When you do, please squash those commits into the earlier ones to reduce the noise.

Although I have requested changes, I am pleased to report that the games do build and work!

<email>totktonada.ru@gmail.com</email>
<name>Alexander Turenko</name>
<email>iansilvester@fastmail.fm</email>
<name>Ian Silvester</name>
Copy link
Member

Choose a reason for hiding this comment

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

You will need to rebase your changes on the latest master as the existing maintainer has already been dropped.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Okay yes, will do.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Currently testing locally. When that's done I'll tackle the rebase and combining the commits to just one, that has the correct headers including sign-off.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Okay, the changes are all in and the commit history cleaned up. I did rebase to latest master, so I don't know why there remains a conflict on metadata.xml. Feel free to remove me as moderator; I am not listed in the file in my repo.

The only other CI warnings are for the copyright message (which we've discussed) and now from the 0.59 ebuild that doesn't recognize the USE flags that the new ebuild introduces.

Looking forward to your feedback @chewi !

@dr-diem
Copy link
Contributor Author

dr-diem commented Jun 17, 2020

I'm withdrawing this PR on the basis that I have combined changes relating to three, albeit inter-related, packages in one commit. I will raise a new one containing three separate 'clean' commits.

@chewi
Copy link
Member

chewi commented Jun 17, 2020

As discussed, remember you can always git push --force over any existing PR with a completely new set of changes.

@dr-diem dr-diem closed this Jun 17, 2020
@dr-diem dr-diem deleted the games-action/dxx-rebirth branch June 17, 2020 16:05
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. no signoff One or more commits do not indicate GCO sign-off.
Projects
None yet
4 participants