Re: [PATCH 1/2] Trivial patch to get the case statement into the coding style of the grub-installer script.
Hi Frank!
On 11/06/2017 10:45 AM, Frank Scheiner wrote:
> case $ARCH in
> powerpc/chrp|powerpc/chrp_rs6k|powerpc/chrp_ibm|powerpc/cell)
> - ;;
> + ;;
> ppc64/chrp|ppc64/chrp_rs6k|ppc64/chrp_ibm|ppc64/cell)
> - ;;
> + ;;
> powerpc/*|ppc64/*)
> - offs=$(findfs /boot/grub)
> - [ -n "$offs" ] || error "GRUB requires that the OF partition is mounted in /boot/grub" 1>&2
> - ;;
> + offs=$(findfs /boot/grub)
> + [ -n "$offs" ] || error "GRUB requires that the OF partition is mounted in /boot/grub" 1>&2
> + ;;
Those are purely formatting changes, so you should also mention
that in the commit message, i.e.:
"Fix formatting for subarchitecture matching for powerpc and ppc64"
It should be clear from the commit message that this patch doesn't
make any functional changes. If you agree, I'll merge the patch as
is, but with the updated commit message.
Adrian
--
.''`. John Paul Adrian Glaubitz
: :' : Debian Developer - glaubitz@debian.org
`. `' Freie Universitaet Berlin - glaubitz@physik.fu-berlin.de
`- GPG: 62FF 8A75 84E0 2956 9546 0006 7426 3B37 F5B5 F913
Reply to: