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

app-arch/lz4: Add USE=lib-only #22168

Closed
wants to merge 1 commit into from
Closed

app-arch/lz4: Add USE=lib-only #22168

wants to merge 1 commit into from

Conversation

jsmolic
Copy link
Member

@jsmolic jsmolic commented Aug 31, 2021

  • Add lib-only flag which provides a minimal lz4 build
    with only library and without other binaries. This is
    useful in e.g. embedded or stripped down systems where
    one might want just the core functionality of liblz4.

Signed-off-by: Jakov Smolić jsmolic@gentoo.org

* Add lib-only flag which provides a minimal lz4 build
  with only library and without other binaries. This is
  useful in e.g. embedded or stripped down systems where
  one might want just the core functionality of liblz4.

Signed-off-by: Jakov Smolić <jsmolic@gentoo.org>
@gentoo-bot
Copy link

Pull Request assignment

Submitter: @jsmolic
Areas affected: ebuilds
Packages affected: app-arch/lz4

app-arch/lz4: @mgorny, @amadio

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.


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. labels Aug 31, 2021
@jsmolic jsmolic requested a review from mgorny August 31, 2021 11:21
@@ -14,7 +14,7 @@ LICENSE="BSD-2 GPL-2"
# https://abi-laboratory.pro/tracker/timeline/lz4/
SLOT="0/r132"
KEYWORDS="~alpha amd64 arm arm64 hppa ~ia64 ~m68k ~mips ppc ppc64 ~riscv ~s390 sparc x86 ~amd64-linux ~x86-linux ~ppc-macos ~x64-macos ~sparc-solaris ~sparc64-solaris ~x64-solaris ~x86-solaris"
IUSE="static-libs"
IUSE="lib-only static-libs"
Copy link
Member

Choose a reason for hiding this comment

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

I'd consider IUSE=minimal instead, as that's an existing global use flag used by many packages.

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm okay with minimal as well

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2021-08-31 11:29 UTC
Newest commit scanned: 260d4f5
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/fab9f2ac42/output.html

@mgorny
Copy link
Member

mgorny commented Aug 31, 2021

What's the end goal here? Given that lz4 executable is 80k in size (+3k for manpage), this sounds like a serious overkill. For a start, it's going to mean that we're going to have to verify all revdeps for whether they're using the library or the executable, and force [-lib-only(-)] on all of them.

This really looks like a case for INSTALL_MASK rather than USE flags. Though in either case you're asking for trouble if you're installing an archiver without its CLI.

@jsmolic
Copy link
Member Author

jsmolic commented Aug 31, 2021

@mgorny Yeah, I'm aware about the size, although it still matters when you're fighting for every kb but I admit difference is small, that's why I was unsure and wanted to hear your opinion. I do have a use case for this, but at the same time I don't want to create more unnecessary work given that difference here is very small and I don't know if we want to go through updating all revdeps just for this. In any case if you're not up for accepting this I can opt for a different solution and avoid changing ::gentoo ebuild

@mgorny
Copy link
Member

mgorny commented Aug 31, 2021

Yes, I'd prefer not having this in ::gentoo. It feels like microoptimizing to me.

@jsmolic
Copy link
Member Author

jsmolic commented Aug 31, 2021

Okay, thanks for your input

@jsmolic jsmolic closed this Aug 31, 2021
@jsmolic jsmolic deleted the lz branch August 31, 2021 16:09
gentoo-bot pushed a commit that referenced this pull request Apr 18, 2024
Changelog:
    Fixed a bug that could leak IPAM entries when a network was removed (#22034).
    Fixed a bug that could cause the rootless network namespace to not be cleaned up on if an error occurred during setup resulting in errors relating to a missing resolv.conf being displayed (#22168).
    Fixed a bug where Podman would use rootless network namespace logic for nested containers (#22218).
    Fixed a bug where writing to volumes on a Mac could result in EACCESS failures when using the :z or :Z volume mount options on a directory with read only files (#19852)

    Fixed a bug in the Compat List endpoint for Networks which could result in a server crash due to concurrent writes to a map (#22330).

Signed-off-by: Rahil Bhimjiani <me@rahil.rocks>
From: #36302
Signed-off-by: Zac Medico <zmedico@gentoo.org>
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.
Projects
None yet
5 participants