Re: [Rpm-maint] [rpm-software-management/rpm] Moving unnecessary notice about empty rpm packages to debug message. (#615)
I think that the patch fixes an ERROR in user interface of rpm utility. When a user asks for a list of package's files he wants to get exactly the list of files, doesn't he? And if there's no files in the package then he wants to get an empty output, doesn't he? Why instead of empty output he gets a text message (according to user's locale)? I think it looks unpredictable and non intuitive. -- 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/615#issuecomment-449499880___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] Moving unnecessary notice about empty rpm packages to debug message. (#615)
For the same reason there is ls when a script uses *. -- 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/615#issuecomment-449497239___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] Moving unnecessary notice about empty rpm packages to debug message. (#615)
> You can get the list of files with rpm -q --qf='[%{FILENAMES}\n]'. There is > nothing dirty about that, those are the building blocks. So, for what `rpm -ql` ? And, as example same behaviour of `ls` (Why you have OWN WAY and why this solution right?): ```Bash $ mkdir empty_dir $ cd empty_dir/ $ ls $ echo $? 0 ``` -- 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/615#issuecomment-449495684___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] Moving unnecessary notice about empty rpm packages to debug message. (#615)
You can get the list of files with rpm -q --qf='[%{FILENAMES}\n]'. There is nothing dirty about that, those are the building blocks. -- 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/615#issuecomment-449483367___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] Moving unnecessary notice about empty rpm packages to debug message. (#615)
> I don't see that this is the rpm binary's issue I completely disagree with this imprisonment. You (or your team ) made strange, unpopular decision that makes problems for simple (popular, expected) case. Other software put this warning to stderr or just return nothing with success code. You forced developers to produce hardcode that forces bugs with different behaviors in environments. Software shouldn't force you to write bad code. -- 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/615#issuecomment-449480485___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] Moving unnecessary notice about empty rpm packages to debug message. (#615)
if the problem is an empty file being queried, why not solve teh problem where the action occurs (before the test), with something simple like: [ -s $ARG ] && ... I don't see that this is the rpm binary's issue -- 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/615#issuecomment-449471367___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] Moving unnecessary notice about empty rpm packages to debug message. (#615)
> You can use rpm -q --qf='%{FILENAMES:arraysize}\n' to get the number of files. Ok. And what about getting a list of included files? Do you have a dirty trick for that? And why we can't just trust `rpm -ql` to do this simple task? -- 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/615#issuecomment-449464112___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] Moving unnecessary notice about empty rpm packages to debug message. (#615)
> changing them _will_ break some scripts that expect them there No, you can't even rely on this behavior in scripts, because this message (contains no files) is locale-dependent. You can't use it. -- 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/615#issuecomment-449462630___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] pythondistdeps: handle ~= version range (#616)
Hmm, so according to [PEP 440](https://www.python.org/dev/peps/pep-0440/#compatible-release), this is equivalent to `(pyramid >= 1.7 with pyramid < 2.0)`. So it's equivalent to `^` in Rust and Ruby. -- 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/issues/616#issuecomment-449429580___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] Moving unnecessary notice about empty rpm packages to debug message. (#615)
You can use rpm -q --qf='%{FILENAMES:arraysize}\n' to get the number of files. -- 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/615#issuecomment-449384859___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
Re: [Rpm-maint] [rpm-software-management/rpm] Moving unnecessary notice about empty rpm packages to debug message. (#615)
This (spewing "helpful" messages into stdout) is a much broader problem than just this one message, and we should address them all at once or not at all. As much as mostly everybody hates these "helpful" messages, changing them *will* break some scripts that expect them there, so if/when breakage is necessary it's better to break clean once than a little bit here in one release and some other bit in the next etc. -- 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/615#issuecomment-449343652___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
[Rpm-maint] [rpm-software-management/rpm] pythondistdeps: (#616)
The Bodhi has requirement written like `pyramid~=1.7`. Unfortunately, pythondistdeps doesn't handle this. It makes RPM to produce following error: ``` error: Dependency tokens must begin with alpha-numeric, '_' or '/': ~= 1.7 ``` -- 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/issues/616___ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint