[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

[pkg-wine-party] Bug#764212: Bug#764212: [wine-development] Error in bash script, fi needed at line 28



severity 764212 grave
found 764212 1.7.28-1
thanks


Raising severity because with this bug the package is not fit for
"testing". Of course "unstable" users may happily install it and use it
after fixing.



On 10/09/2014 10:46 AM, Konstantin Demin wrote:
> Please review attached patch.

Most parts of your patch aren't related to this bug but simply use
another style (replacing spaces with tabs; using bash builtins [ ]
instead of test (I would prefer that too); omitting "&&" (bad idea)).

As already noted previously the patch for this bug is really just adding
the missing "fi" (haven't tested it, but it seems obvious).

Please search the web how to submit correct patches. Basically you
should keep them minimal and adapt to the current style of the file that
you want to patch.

If you want other things changed in wine file another bug with your
patch and describe what you did there for what purpose.

Greets
jre



Reply to: