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

gimp: 2.10.38 → 3.0.0-RC2 #67576

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from
Draft

gimp: 2.10.38 → 3.0.0-RC2 #67576

wants to merge 8 commits into from

Conversation

jtojnar
Copy link
Member

@jtojnar jtojnar commented Aug 27, 2019

Trying to see if we need any changes on Darwin.

Depends on #73833

closes #333800

@jtojnar jtojnar added the 2.status: wait-for-upstream Waiting for upstream fix (or their other action). label Sep 9, 2019
@jtojnar jtojnar mentioned this pull request Oct 31, 2019
1 task
@jtojnar
Copy link
Member Author

jtojnar commented Nov 5, 2019

The gjs failure will require #72805

@ofborg ofborg bot added 11.by: package-maintainer This PR was created by the maintainer of the package it changes 10.rebuild-darwin: 1-10 10.rebuild-linux: 11-100 labels Nov 5, 2019
];

buildInputs = [
python3
Copy link
Contributor

@worldofpeace worldofpeace Nov 5, 2019

Choose a reason for hiding this comment

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

Think this won't work if it ends up in a shebang for darwin (because wrappers).

Copy link
Member Author

Choose a reason for hiding this comment

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

Bah. Hoping for C wrappers before GIMP 3 is released.

Copy link
Member

Choose a reason for hiding this comment

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

Good news! makeBinaryWrapper is a thing now: #124556

@jtojnar jtojnar force-pushed the gimp-meson branch 2 times, most recently from 715ab15 to f52c409 Compare November 20, 2019 22:02
@ofborg ofborg bot added the 2.status: merge conflict This PR has merge conflicts with the target branch label Nov 20, 2019
@jtojnar jtojnar changed the title gimp: switch to unstable gimp: Testing dev version with Meson Nov 21, 2019
@ofborg ofborg bot removed the 2.status: merge conflict This PR has merge conflicts with the target branch label Nov 21, 2019
@jtojnar jtojnar force-pushed the gimp-meson branch 2 times, most recently from c750da1 to 359513f Compare November 21, 2019 22:58
@jtojnar
Copy link
Member Author

jtojnar commented Dec 13, 2019

@GrahamcOfBorg eval

@jtojnar jtojnar mentioned this pull request Nov 7, 2024
13 tasks
@Eveeifyeve
Copy link
Contributor

Is this PR abandoned?

@lf-
Copy link
Member

lf- commented Dec 15, 2024

Is this PR abandoned?

i don't expect so, it's more that it probably should be in a different format while gnu imp 3.x is still in dev

also btw there's a bug where the meson build type is wrong so there's no optimization and the performance sucks

@github-actions github-actions bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin and removed 10.rebuild-darwin: 1-10 labels Dec 27, 2024
@jtojnar jtojnar changed the title gimp: Testing dev version with Meson gimp: 2.10.38 → 3.0.0-RC1 Dec 27, 2024
@jtojnar
Copy link
Member Author

jtojnar commented Dec 27, 2024

Is this PR abandoned?

I update it as time permits. Just bumped it to 3.0.0-RC1.

also btw there's a bug where the meson build type is wrong so there's no optimization and the performance sucks

How do you determine that? We use the plain meson build type so the optimization flags from stdenv should apply. This should be -O2 by default IIRC.

Edit: Yeah, default-enabled fortify hardening does -O2, which should be the same as the debugoptimized mode of Meson, which GIMP defaults to.

@ofborg ofborg bot removed the 2.status: merge conflict This PR has merge conflicts with the target branch label Dec 27, 2024
@github-actions github-actions bot added 10.rebuild-darwin: 11-100 and removed 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin labels Dec 27, 2024
@ofborg ofborg bot requested a review from AndersonTorres December 27, 2024 19:45
@jtojnar jtojnar changed the title gimp: 2.10.38 → 3.0.0-RC1 gimp: 2.10.38 → 3.0.0-RC2 Dec 28, 2024
@jtojnar jtojnar mentioned this pull request Dec 28, 2024
13 tasks
@Eveeifyeve Eveeifyeve added the 2.status: merge conflict This PR has merge conflicts with the target branch label Dec 28, 2024
@Eveeifyeve
Copy link
Contributor

Please rebase pr with merged changes from #368691.

@ofborg ofborg bot removed the 2.status: merge conflict This PR has merge conflicts with the target branch label Dec 30, 2024
Copy link
Contributor

@Eveeifyeve Eveeifyeve left a comment

Choose a reason for hiding this comment

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

Please fix commit message of 771cf to be accordance of CONTRIBUTING.md

@emilazy
Copy link
Member

emilazy commented Dec 30, 2024

@Eveeifyeve That is a fixup commit that will be squashed into the previous commit when this is ready for merge, as is common Git practice. I appreciate you’re enthusiastic about helping out, but as previously requested I’d like to suggest you please avoid leaving review feedback for things you don’t fully understand until you’ve gained more experience in Nixpkgs, as it takes up the time of experienced contributors like @jtojnar and anyone who is subscribed to this PR.

@wegank wegank added the 2.status: merge conflict This PR has merge conflicts with the target branch label Feb 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.status: merge conflict This PR has merge conflicts with the target branch 2.status: wait-for-upstream Waiting for upstream fix (or their other action). 10.rebuild-darwin: 11-100 10.rebuild-linux: 11-100 11.by: package-maintainer This PR was created by the maintainer of the package it changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.