Now that libgweather ships the xml split per language (yay !), it would be a nice touch to add proper %lang tags to the file list, so that hypothetical localized spins can reap the benefits of the split. Actually, it may already benefit our regular livecd spin, since that also uses a reduced list of supported languages.
Created attachment 314827 [details] %lang'ify Locations.*.xml Incidentally, I did this about a week ago, see attached patch. It also contains an unrelated hunk which moves xsltproc from %install to %build where it is IMO better placed, as well as uses xmllint --noblanks to do the whitespace reduction which also feels better than editing XML with sed (although it's pretty trivial in this case).
Ah, cool. I tried to do this earlier in the week, but no cvs... I don't really know rpmspec-ese, so I can't say if this is right, although just looking at the diff, I think this part: +xsltproc %{SOURCE1} data/Locations.xml > data/loc.xml +xmllint --noblanks data/loc.xml > data/Locations.xml (which you moved from install-time to make-time) is irrelevant now, because libgweather does --noblanks itself in 2.24.
Seems so indeed, already in 2.23.6.
Matthias committed this a while ago