Hi Igor,<br><br>Thanks for the patch.  It looks good to me.  I've applied it to trunk.<br><br>Can you please update the documentation for the next version on the wiki to describe the new argument value?<br><a href="http://wiki.openstreetmap.org/wiki/Osmosis/Detailed_Usage_0.39">http://wiki.openstreetmap.org/wiki/Osmosis/Detailed_Usage_0.39</a><br>
<br>Cheers,<br>Brett<br><br><div class="gmail_quote">On Sun, Feb 6, 2011 at 4:12 AM, Igor Podolskiy <span dir="ltr"><<a href="mailto:igor.podolskiy@vwi-stuttgart.de">igor.podolskiy@vwi-stuttgart.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">Hello everybody,<br>
<br>
first of all, thank you all for developing Osmosis! :)<br>
<br>
I currently have some issues with the compressionMethod parameter in the XML tasks. This is just a minor annoyance but nevertheless:<br>
<br>
The compressionMethod parameter of the --[fast]-read-xml task is currently handled somewhat differently than most other parameters in Osmosis. CompressionMethod has three official values (none, gzip, bzip2), but actually there is a fourth option (autodetect) which can only be specified implicitly by omitting compressionMethod altogether.<br>

<br>
Although this is documented in the wiki, it creates problems for people who build tools around Osmosis which read and/or generate Osmosis command lines automatically, and even for a human user, this is another special case which you need to think of (no other parameter in any task works this way).<br>

<br>
This can easily be fixed by creating an explicit fourth value (say, "auto") for compressionMethod, and making it the default value. This change doesn't break old scripts and streamlines the parameters for new ones. The attached patch against current trunk does this (and the trunk still passes all unit tests with the patch applied).<br>

<br>
It would be cool if you could integrate this into the next version of Osmosis. If you have any questions or if the patch needs some improvement, I'll be here on the mailing list :).<br>
<br>
Best regards<br><font color="#888888">
Igor<br>
</font><br>_______________________________________________<br>
osmosis-dev mailing list<br>
<a href="mailto:osmosis-dev@openstreetmap.org">osmosis-dev@openstreetmap.org</a><br>
<a href="http://lists.openstreetmap.org/listinfo/osmosis-dev" target="_blank">http://lists.openstreetmap.org/listinfo/osmosis-dev</a><br>
<br></blockquote></div><br>