https://github.com/apache/maven-shade-plugin/blob/25e7195b6bff41c33b50986a32459a0df0706721/src/main/java/org/apache/maven/plugins/shade/resource/properties/SortedProperties.java ;)
Le jeu. 26 août 2021 à 15:00, Gary Gregory <garydgreg...@gmail.com> a écrit : > Just FYI: > > https://commons.apache.org/proper/commons-collections/apidocs/org/apache/commons/collections4/properties/SortedProperties.html > > Gary > > > On Wed, Aug 25, 2021, 03:20 Michael Osipov <micha...@apache.org> wrote: > > > Correct. The average use does not care about sorting, so he won't be > > affected by a constantly sorted output. They could remove date already > > because it adds zero benefit to the output. > > > > I like your idea with the property to restore compat. > > > > Am 2021-08-25 um 09:13 schrieb Romain Manni-Bucau: > > > Guess it is only useful if it is done in store() methods and not a new > > one > > > if we want to leverage it before >=8 years (2 more lts + a few) since > API > > > cant change for j8. > > > Means the jvm could get a system property to disable the comments and > > sort > > > the keys and a new toggle on the properties object to do the same > > > programmatically per instance. > > > > > > That said Im pretty confident it will not help maven. Only useful thing > > is > > > the escaping of props and it should be trivial to extract and reuse in > a > > > collector on j8 instead of using mjar or reflections which both have > more > > > cons and need the previous option anyway. > > > > > > > > > Le mer. 25 août 2021 à 08:47, Michael Osipov <micha...@apache.org> a > > écrit : > > > > > >> Am 2021-08-24 um 16:24 schrieb Robert Scholte: > > >>> FYI: A discussion has started on core-libs-dev mailinglist[1] about > > >>> extending java.util.Properties with methods to store properties > files > > >>> in a reproducible way. > > >>> If you have any comments, subscribe to the mailinglist[2] or ask me > to > > >>> drop the comments. > > >> > > >> * Date comment must go forever > > >> * Using Optional is purely overengineered, null is more than > sufficient > > >> * Shouldn't it be "List<String> comments" for multiline comments? > > >> * A comparator is a nice to have, but not a must. If it is trivial to > > >> implement, why not? > > >> > > >> Finally, unless this is going to land in 11 or 8 as a backport, we > won't > > >> be able to use it in the next 5 years or so... > > >> > > >> M > > >> > > >> > > >> --------------------------------------------------------------------- > > >> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org > > >> For additional commands, e-mail: dev-h...@maven.apache.org > > >> > > >> > > > > > > > > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org > > For additional commands, e-mail: dev-h...@maven.apache.org > > > > >
