Merged #1392 into master.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/1392#event-3892043957___
Rpm-maint mailing list
Rpm-mai
Since there's no further feedback...
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/1392#issuecomment-711832987___
Rpm-maint mai
@pmatilai pushed 1 commit.
5125725009f5ad50adf5a574b9e9f2fac16af13b fixup! Wrap getopt() usage into
internal helper
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/rpm-software-management/rpm/pull/1392/files/359b380a026b35d8f9fe27c66a
Oh, another good point. I probably had some vague idea about letting the
callback handle errors if it wants, but it wasn't all that carefully thought
out as you can see :sweat_smile:
I'll drop optopt from being passed, the current implementation doesn't need it
and this is a private helper func
Btw, is there a need to pass optopt? It's usually used to print an error
message for unrecognized options, but in that case the callback is not called
anyway. (It *is* called if your option string starts with ':' and there is no
argument to an option that needs one, though.)
--
You are receivi
Fixed to pass optopt around instead of opterr.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/1392#issuecomment-708228238___
Rpm
Yeah, thanks for spotting. Will fix.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/1392#issuecomment-708172511___
Rpm-maint mai
I think you mean optopt instead of opterr, because opterr is for configuring
getopt()'s error handling.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/1392#issuecomment-70
Macros might want to pass along options meant for others while perhaps
modifying some of them, without exposing it all to users.
Have "-" as the parametric macro opts string disable all options
processing in rpm, allowing the macro to handle the raw argument list as they
place.
Fixes: