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-libs/lib3mf 2.1.0: version bump #19412

Closed
wants to merge 4 commits into from
Closed

Conversation

waebbl
Copy link
Contributor

@waebbl waebbl commented Feb 11, 2021

Closes: https://bugs.gentoo.org/769275
Package-Manager: Portage-3.0.14, Repoman-3.0.2
Signed-off-by: Bernd Waibel waebbl-gentoo@posteo.net

Current versions of media-gfx/openscad don't work with this API version and need to be restricted to <lib3mf-2. The upcoming openscad version bump does work with it. See
Bug: https://bugs.gentoo.org/769278

@waebbl waebbl marked this pull request as draft February 11, 2021 08:06
@gentoo-bot
Copy link

Pull Request assignment

Submitter: @waebbl
Areas affected: ebuilds
Packages affected: media-gfx/openscad, media-libs/lib3mf

media-gfx/openscad: waebbl-gentoo[at]posteo.net, @gentoo/proxy-maint
media-libs/lib3mf: waebbl-gentoo[at]posteo.net, @gentoo/proxy-maint

Linked bugs

Bugs linked: 769275


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 Feb 11, 2021
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2021-02-11 08:20 UTC
Newest commit scanned: f64a67d
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/3717101b3e/output.html

@waebbl waebbl marked this pull request as ready for review February 13, 2021 09:49
@waebbl
Copy link
Contributor Author

waebbl commented Feb 13, 2021

Switching openscad live ebuild to use cmake instead of qmake. Works with both, lib3mf-1.8.1 and lib3mf-2.1.0. Latter needs a patch due to a case change from lib3MF.pc to lib3mf.pc.

@waebbl
Copy link
Contributor Author

waebbl commented Feb 13, 2021

For the act dependency needed to re-add keywords for x86 and arm see #19411

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2021-02-13 10:25 UTC
Newest commit scanned: 8aade9e
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/a023dd9f6e/output.html

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2021-03-01 23:30 UTC
Newest commit scanned: 928430f
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/62a2119fd8/output.html

@juippis
Copy link
Member

juippis commented Mar 6, 2021

Is the act dependency still needed? Can we somehow combine this PR with #19714?

@juippis
Copy link
Member

juippis commented Mar 6, 2021

Actually could you combine all the related work to a single PR. I'm at loss, and am fearing there'll be tree breakage or unnecessary rebuilds for users. Openscad is not a little package.

@waebbl
Copy link
Contributor Author

waebbl commented Mar 6, 2021

@juippis Sure, I can combine them in a single PR.

The act package is 'only' needed to get x86 and arm keywords back to lib3mf. For amd64 the included binary does the job. Should I add it to the PR too?

@juippis
Copy link
Member

juippis commented Mar 6, 2021

Up to you, I quickly glanced at the ebuild and it might need .. a lot of fixes before it can be pushed. So maybe best to leave it out for now.

@waebbl
Copy link
Contributor Author

waebbl commented Mar 6, 2021

Yes it might need some rework. The point with the package is, that upstream doesn't follow the usual way Go packages are developed, so the Go eclasses don't seem to be useful. On the other hand the build is rather trivial, so I decided for the way it's done.
I agree, it's better to leave it out for now and wait for a dedicated review of the package.

@juippis
Copy link
Member

juippis commented Mar 6, 2021

I've nearly caught up with our PR work queue, please ping me in ~a week from now for the act package and I'll take a look.

Closes: https://bugs.gentoo.org/769275
Package-Manager: Portage-3.0.14, Repoman-3.0.2
Signed-off-by: Bernd Waibel <waebbl-gentoo@posteo.net>
This package version does not build against lib3mf-2 API.

Package-Manager: Portage-3.0.14, Repoman-3.0.2
Signed-off-by: Bernd Waibel <waebbl-gentoo@posteo.net>
Now uses cmake instead of qmake.

Package-Manager: Portage-3.0.14, Repoman-3.0.2
Signed-off-by: Bernd Waibel <waebbl-gentoo@posteo.net>
Bug: https://bugs.gentoo.org/773217
Closes: https://bugs.gentoo.org/769278
Package-Manager: Portage-3.0.16, Repoman-3.0.2
Signed-off-by: Bernd Waibel <waebbl-gentoo@posteo.net>
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2021-03-06 23:50 UTC
Newest commit scanned: 5ed5294
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/ff764ade8e/output.html

Copy link
Member

@juippis juippis left a comment

Choose a reason for hiding this comment

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

Let me know the answers to my questions, there's no need for you to do anything, I can fix those minor issues while merging.

@@ -46,7 +46,7 @@ RDEPEND="
media-libs/freetype
>=media-libs/glew-2.0.0:0=
media-libs/harfbuzz:=
media-libs/lib3mf
<media-libs/lib3mf-2
Copy link
Member

Choose a reason for hiding this comment

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

Will the update cause runtime issues? Ie should this be revbumped for this...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

2019.05 will not build with lib3mf-2. Lib3mf-2 has been released after 2019.05 and support for it has not been backported

Copy link
Member

Choose a reason for hiding this comment

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

The idea is, people who have 2019.05-r4 installed now will update to lib3mf-2, because the restriction won't hit their package database before a rebuild. So, if lib3mf-2 will cause runtime issues to openscad-2019.05, this ebuilds needs t obe revbumped so that won't happen.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

As the package won't build with lib3mf-2 I suspect it will cause runtime issues, whenever someone tries to im-/export a 3mf format file.
I'm gonna revbump the package then.

media-libs/freetype
>=media-libs/glew-2.0.0:0=
media-libs/harfbuzz:=
media-libs/lib3mf
Copy link
Member

Choose a reason for hiding this comment

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

Does this work with both 1 and 2? Is some restriction needed? That's also why I asked to combine the PRs so that no modification is needed for this version after merging.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The new version works with both lib3mf versions.

@gentoo-bot gentoo-bot closed this in 764dd0f Mar 7, 2021
@waebbl waebbl deleted the lib3mf-2.1.0 branch March 7, 2021 14:04
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