Currently, the 'fields' argument to make_desktop_entry() has to be specified as printf format string. I guess this is because it is supposed to support escape sequences like '\n'. This has the disadvantage that additionally all occurences of '%' have to be escaped. As it's impossible to pass any additional args to that printf invocation, I don't see a reason why we should require the arg being a printf format string. We could use '%b' conversion specifier instead, which does parse all escape sequences in the argument. This way is more formally corect. And as the possibility of having such '%'-chars in current 'fields' arg values tends to zero and the help message lacks the explicit warning about having to escape them, I guess it should be feasible to change this behaviour without keeping backwards compatibility.
i'm guessing you're suggesting: - [[ -n ${fields} ]] && printf "${fields}\n" >> "${desktop}" + [[ -n ${fields} ]] && printf '%b\n' "${fields}" >> "${desktop}" that should be fine
Yes, that's what I meant.
done then, thanks http://sources.gentoo.org/eclass/eutils.eclass?r1=1.341&r2=1.342
I guess you should update the description to be more precise too.
meh
