Community
Participate
Working Groups
In pdebuild.sh, we do a lot of ls *blah*. It would be nice to pipe the output of this to /dev/null and react to non-zero return codes. ls *blah* 2> /dev/null, etc.
Created attachment 203120 [details] Patch proposition should the execution of the script also be stopped when ls -d return an error? The error seems to just say that nothing matches "optional" in particular location - but since it is based on wildcards, you do not really know if it is an error or maybe it works as designed. I thought about checking if there is at least one match in all locations, but I am not sure if this is a right thing to do.
(In reply to comment #1) > Created attachment 203120 [details] > Patch proposition > > should the execution of the script also be stopped when ls -d return an error? > > The error seems to just say that nothing matches "optional" in particular > location - but since it is based on wildcards, you do not really know if it is > an error or maybe it works as designed. > > I thought about checking if there is at least one match in all locations, but I > am not sure if this is a right thing to do. Thanks for looking into this Christopher, ideally one would test it on Fedora with a few plugins to verify that no corner case is broken. I'll try to test it this week too and either apply or report a problem.
Applied in master 1eb6f9e..ae6a179. Thanks again.
