Thinking out loud, we can do it already using a classfiletransformer in
classrealm for free for mojos, no?

Le jeu. 26 août 2021 à 15:08, Romain Manni-Bucau <rmannibu...@gmail.com> a
écrit :

>
> 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
>> >
>> >
>>
>

Reply via email to