Temurin-build: Facilitez l'ajout d'indicateurs supplémentaires sur makejdk

Créé le 13 juin 2017  ·  6Commentaires  ·  Source: adoptium/temurin-build

Nous devrions être en mesure de passer trivialement tous les arguments de configuration supplémentaires

Voir http://mail.openjdk.java.net/pipermail/distro-pkg-dev/2015-June/032392.html qui mentionne -with-extra-cflags, --with-extra-cxxflags et --with-extra -ldflags

Je ne devrais pas avoir besoin de modifier sbin/build.sh pour ajouter des arguments de configuration supplémentaires

quelque chose comme makejdk-any-platform.sh -copts

enhancement

Tous les 6 commentaires

Je pense que export CONFIGURE_ARGS_FOR_ANY_PLATFORM="" déjà ça

ajouté cette var à la page de manuel https://github.com/AdoptOpenJDK/openjdk-build/pull/125

@gdams merci d'avoir repris cela - je pense que documenter cela signifierait moins ou pas de requêtes dans ce domaine. Une fois la documentation en place, je pense que nous n'avons pas besoin d'aborder cela comme un problème.

Sauf si nous repérons d'autres indicateurs qui pourraient changer afin qu'ils puissent être transmis via CLI.

makejdk-any-platform.sh a des drapeaux codés en dur pour les différentes plates-formes car nous (sur les conseils de @tellison) voulions pouvoir avoir un et un seul moyen de construire l'OpenJDK soit sur une machine locale, soit sur un système CI pour toute plate-forme spécifique qui toujours travaillé pour tout le monde - nous devrions continuer ainsi. Pour les cas particuliers, si les utilisateurs souhaitent toujours outrepasser ces indicateurs, ils peuvent désormais très bien le faire à l'aide de la documentation.

Je suppose que nous pourrions permettre aux utilisateurs d'ajouter plus facilement des drapeaux tant que notre version Jenkins ne change pas. pensées de @tellison ?

Je pense que https://github.com/AdoptOpenJDK/openjdk-build/commit/74bdae78bacc9472d7c1a9879da1a9ef21b61854 a résolu ce problème avec un nouveau paramètre `-ca``, fermant donc

Cette page vous a été utile?
0 / 5 - 0 notes

Questions connexes

ChristianCiach picture ChristianCiach  ·  7Commentaires

mstoodle picture mstoodle  ·  3Commentaires

sophia-guo picture sophia-guo  ·  6Commentaires

pshipton picture pshipton  ·  4Commentaires

nebhale picture nebhale  ·  7Commentaires