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

gnu-getopt: new recipe #23537

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

gnu-getopt: new recipe #23537

wants to merge 6 commits into from

Conversation

valgur
Copy link
Contributor

@valgur valgur commented Apr 15, 2024

Packages the GNU variant of the getopt command-line utility. Not to be confused with the getopt library already available on CCI.

Required on macOS for the shell scripts included in the f2c PR #23417.

The current version of the tool was originally hosted at http://frodo.looijaard.name/project/getopt but has now been merged into linux-utils. There's a slight question of whether we should simply build and package all of the utils in linux-utils instead. However, I don't think they are all compatible with other OS-s so it's safer to package getopt in isolation for now.

Packaging status

@conan-center-bot

This comment has been minimized.

@jwillikers
Copy link
Contributor

Ugh, this will mean another separate project to convert to Meson. 😢 I just got changes in the stable branch for 2.40 that should make it possible to build this with Meson on macOS, but it looks like you're also supporting Windows. I'd better see if Meson needs any changes to support that.

@jwillikers
Copy link
Contributor

Fyi, I've opened issue util-linux/util-linux#2962 to make it possible to build with Meson on Windows without requiring /bin/sh.

Copy link
Contributor

@jwillikers jwillikers left a comment

Choose a reason for hiding this comment

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

LGTM! I'd still like to lump all of the util-linux recipes together in one recipe eventually.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline ✔️

All green in build 4 (d665480b7d6c0b2a3fc3dd71b81b91865ba47ad0):

  • gnu-getopt/2.40:
    All packages built successfully! (All logs)

Conan v2 pipeline ✔️

Note: Conan v2 builds are now mandatory. Please read our discussion about it.

All green in build 4 (d665480b7d6c0b2a3fc3dd71b81b91865ba47ad0):

  • gnu-getopt/2.40:
    All packages built successfully! (All logs)

Copy link
Member

@AbrilRBS AbrilRBS left a comment

Choose a reason for hiding this comment

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

Thanks! Minor question regarding the sqlite3 dependency, other than that, looks good :)

recipes/gnu-getopt/all/conanfile.py Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants