Effettivamente è un bug. Ma non essendomene mai accorto.... ;)

Ci sono una serie di quirk di bash di cui sono al corrente (tipo la
differenza tra "[" e "[[") o gli apici singoli e doppi, e anche $* e $@.
Prendo nota come best practice di usare "$@".

Lo correggo quando arrivo a casa. ;)

Grazie.
--Luigi



On Wed, May 8, 2019 at 10:59 PM Paolo Mossino [email protected]
[it-torino-java-jug] <[email protected]> wrote:

>
>
> Concorderei, MA:
>
>    - stiamo parlando di un dotfiles, quindi mi aspetto sia l'autore a
>    considerare e decidere cosa farci, visto che è teoricamente a suo uso e
>    consumo semi-esclusivo ;-)
>    - è possibile che A) funzioni per via compensazioni altrove (tipo
>    doppio bug che si annullano a vicenda) o B) l'autore faccia leva su questo
>    comportamento per fare magie, quindi risolverlo potrebbe impattare
>    dipendenze esterne di cui non sono a conoscenza
>
> Parliamo di Java, ecco un esempio di magie che funzionicchiano sfruttando
> il fatto che *NON* hai quotato correttamente le variabili (e quindi
> chiedi alla bash di fare un parsing al volo):
>
> OPTS="-Dfoo=bar -Daaa=bbb"
> java ${OPTS} "$@"
>
>
> La versione corretto richiede di usare un array bash (se la version di
> Bash lo supporta):
>
> declare -a OPTS=(
>     '-DDfoo=bar'
>     '-Daaa=bbb'
> )
>
> java "${OPTS[@]}" "$@"
>
>
> .... ma questo non funziona bene se OPTS veniva dall'environment (che
> AFAICR non supporta gli array, solo stringhe) anziché da un file sourced.
>
> Un *buon* esempio di come fare uno startup di un programma Java
> correttamente l'ho trovato in jetty.sh... good job! ;-)
>
>
>
> On Wed, May 8, 2019 at 3:38 PM Roberto Franchini [email protected]
> [it-torino-java-jug] <[email protected]> wrote:
>
>>
>>
>>
>>
>> On Wed, May 8, 2019 at 3:33 PM Paolo Mossino [email protected]
>> [it-torino-java-jug] <[email protected]> wrote:
>>
>>>
>>>
>>>  [cut]
>>
>>
>>
>>> L'errore appare nei vari wrapper, tipo
>>>
>>>    - https://github.com/lviggiano/dotfiles/blob/master/bin/java10#L2
>>>    - https://github.com/lviggiano/dotfiles/blob/master/bin/__java#L41
>>>
>>>
>>>
>> PR subito!!!!!
>> That's the way!
>>
>> FRANK
>> --
>> Roberto Franchini
>> "The impossible is inevitable"
>> https://github.com/robfrank/
>> https://twitter.com/robfrankie
>> https://www.linkedin.com/in/robfrank
>>
>>
>
> --
> Paolo Mossino  <[email protected]>
>
> 
>

Reply via email to