[pacman-dev] pacman shell scripts and bash3.2 compat - Was: [PATCH 04/11] makepkg: less code duplication in create_package
On Sun, Jun 20, 2010 at 6:09 AM, Allan McRae
That is a bash4 command so will break cygwin compatibility (which currently has bash-3.2). Resubmit with something like "tr [:lower:] [:upper:]" usage.
(I know the ${i^^} is used elsewhere, but that slipped passed my notice and will be being reverted.
bash_completion also has bash 4 only items... Please write a style guide for all shell scripts.... Andres P
On 21/06/10 03:21, Andres P wrote:
On Sun, Jun 20, 2010 at 6:09 AM, Allan McRae
wrote: That is a bash4 command so will break cygwin compatibility (which currently has bash-3.2). Resubmit with something like "tr [:lower:] [:upper:]" usage.
(I know the ${i^^} is used elsewhere, but that slipped passed my notice and will be being reverted.
bash_completion also has bash 4 only items...
Can you point them out?
Please write a style guide for all shell scripts....
Yes, _someone_ should... I have put it low down on my ever growing TODO list. Allan
To avoid errors with bash 3.2, compopt will be skipped if it's not a
shell builtin.
compopt is needed to not append slashes to package names that
coincide with directories in PWD.
This is currently not possible to fix in bash versions that do not support
compopt, so these users will have to bear that regression.
Signed-off-by: Andres P
On 22/06/10 12:30, Andres P wrote:
To avoid errors with bash 3.2, compopt will be skipped if it's not a shell builtin.
compopt is needed to not append slashes to package names that coincide with directories in PWD.
This is currently not possible to fix in bash versions that do not support compopt, so these users will have to bear that regression.
Signed-off-by: Andres P
Signed-off-by: Allan On my working-maint branch.
participants (2)
-
Allan McRae
-
Andres P