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-gfx/inkscape: bump to 1.1 & sync live #22353

Closed
wants to merge 2 commits into from

Conversation

arkamar
Copy link
Member

@arkamar arkamar commented Sep 21, 2021

This PR synces live ebuild with upstream and bunps inkscape to 1.1, where I also backported patch from upstream in order to support musl.

I enabled testing in live ebuild, where it works for me, but it is still restricted for 1.1 because few tests fail and it would be necessary to backport multiple quite big commits to make it work, which didn't make sense to me.

I am not sure if https://bugs.gentoo.org/show_bug.cgi?id=776265 should be closed as well.

- Add optional readline support,
- gtkspell dependency was changed to gspell and aspell is not required
  anymore,
- testing

Closes: https://bugs.gentoo.org/790269
Signed-off-by: Petr Vaněk <arkamar@atlas.cz>
@gentoo-bot
Copy link

Pull Request assignment

Submitter: @arkamar
Areas affected: ebuilds
Packages affected: media-gfx/inkscape

media-gfx/inkscape: @Zlogene, @akhuettel

Linked bugs

Bugs linked: 790269, 791925


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). bug linked Bug/Closes found in footer, and cross-linked with the PR. labels Sep 21, 2021
@arkamar arkamar changed the title media-gfs/inkscape: bump to 1.1 & sync live media-gfx/inkscape: bump to 1.1 & sync live Sep 21, 2021
@arkamar
Copy link
Member Author

arkamar commented Sep 21, 2021

FP where ~riscv is restored because it is present in 1.0.2-r2, but I removed ~hppa because app-text/gspell dependency does not have it as well.

@arkamar
Copy link
Member Author

arkamar commented Sep 21, 2021

FP with dropped ~ia64 keyword bacause of the same reason as above, app-text/gspell dependency does not have it.

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2021-09-21 15:40 UTC
Newest commit scanned: 943f42f
Status: ✅ good

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/6eea4395e5/output.html


EAPI=7

PYTHON_COMPAT=( python3_{8..9} )

Choose a reason for hiding this comment

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

Do you really need an interval for only 2 python versions here?

Copy link
Member Author

@arkamar arkamar Sep 22, 2021

Choose a reason for hiding this comment

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

No, I don't need it. This is used more often than {8,9} in ::gentoo.

Copy link
Member Author

Choose a reason for hiding this comment

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

Anyway, do you want me to change it?

Copy link
Member Author

Choose a reason for hiding this comment

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

Changed to {8,9} in recent FP.

- Add optional readline support
- gtkspell dependency was changed to gspell, aspell is not required
  anymore
- backported sentinel patch from upstream in order to support musl libc
- dropping ~hppa and ~ia64 keyword because they are missing in
  app-text/gspell dependency

Closes: https://bugs.gentoo.org/791925
Signed-off-by: Petr Vaněk <arkamar@atlas.cz>
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2021-09-23 08:20 UTC
Newest commit scanned: 088a896
Status: ✅ good

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

@arkamar arkamar deleted the inkscape branch September 25, 2021 16:18
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). bug linked Bug/Closes found in footer, and cross-linked with the PR.
Projects
None yet
4 participants