RFR: 8347755: Support static library in jmod [v4]

Henry Jen henryjen at openjdk.org
Wed Mar 5 15:28:26 UTC 2025


On Wed, 5 Mar 2025 13:29:35 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:

>> Henry Jen has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   support major version validation
>
> src/java.base/share/classes/jdk/internal/jmod/JmodFile.java line 195:
> 
>> 193:         } else {
>> 194:             os.write(JMOD_MAGIC_NO_STATIC_LIB);
>> 195:         }
> 
> This conditionality seems a bit strange. Isn't the idea for the magic to be changed when the format got extended to do more things? I.e. shouldn't this be micro version `0x01` in both cases? Then when dealing with version JMOD `1.1` version you'd be checking if static-libs is there.

The idea is without static lib, it's essential the same as the old format, using the old version so that old tools can be used if static is not included.

-------------

PR Review Comment: https://git.openjdk.org/leyden/pull/46#discussion_r1981636419


More information about the leyden-dev mailing list