| Summary: | Localepurge parses options badly | ||
|---|---|---|---|
| Product: | Gentoo Linux | Reporter: | Charlie Shepherd (RETIRED) <masterdriverz> |
| Component: | New packages | Assignee: | José Alberto Suárez López (RETIRED) <bass> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | ||
| Priority: | High | ||
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
| Attachments: | Patch to change the way localpurge parses options | ||
Created attachment 90077 [details, diff]
Patch to change the way localpurge parses options
Thanks :) nice patch, 0.5.1 now in cvs |
Localepurge currently parses options like this if [ "$1" = "-debug" ] || [ "$1" = "-d" ] \ || [ "$2" = "-debug" ] || [ "$2" = "-d" ] \ || [ "$3" = "-debug" ] || [ "$3" = "-d" ] \ || [ "$4" = "-debug" ] || [ "$4" = "-d" ] \ || [ "$5" = "-debug" ] || [ "$5" = "-d" ]; then set -x fi So if I (ok so this is unlikely) execute localepurge -nc -nc -nc -nc -nc -debug it won't find debug. A better way of doing it is for x in $@ do if "$x" = "-debug" ] || [ "$x" = "-d"; then set -x fi done See the attached patch for more comprehensive rewrite.