Skip to content

Conversation

csfore
Copy link
Contributor

@csfore csfore commented Aug 9, 2024

  • Tests pass
  • Patched file is not an installed header

Note: Upstream PR fixes an additional file that does not error on 2.2.0
Upstream: 3MFConsortium/lib3mf#387
Closes: https://bugs.gentoo.org/937423


Please check all the boxes that apply:

  • I can submit this contribution in agreement with the Copyright Policy.
  • I have certified the above via adding a Signed-off-by line to every commit in the pull request.
  • This contribution has not been created with the assistance of Natural Language Processing artificial intelligence tools, in accordance with the AI policy.
  • I have run pkgcheck scan --commits --net to check for issues with my commits.

Please note that all boxes must be checked for the pull request to be merged.

- Tests pass
- Patched file is not an installed header

Note: Upstream PR fixes an additional file that does not error on 2.2.0
Upstream: 3MFConsortium/lib3mf#387
Closes: https://bugs.gentoo.org/937423
Signed-off-by: Christopher Fore <csfore@posteo.net>
@gentoo-bot
Copy link

Pull Request assignment

Submitter: @csfore
Areas affected: ebuilds
Packages affected: media-libs/lib3mf

media-libs/lib3mf: @negril, @gentoo/proxy-maint

Linked bugs

Bugs linked: 937423


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 Aug 9, 2024
@gentoo-bot gentoo-bot closed this in 9f442ca Aug 9, 2024
@@ -0,0 +1,13 @@
diff --git a/Source/Common/Platform/NMR_ImportStream_Unique_Memory.cpp b/Source/Common/Platform/NMR_ImportStream_Unique_Memory.cpp
Copy link
Member

Choose a reason for hiding this comment

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

In future, include the Gentoo bug link + PR link in the patch itself at the top as well in case other distros pick it up, so they have provenance.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will do!

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
Development

Successfully merging this pull request may close these issues.

3 participants