RFR: 8345874: Run make doctor automatically on failed CI builds [v2]
Magnus Ihse Bursie
ihse at openjdk.org
Mon Apr 28 09:44:48 UTC 2025
On Sun, 20 Apr 2025 04:01:31 GMT, SendaoYan <syan at openjdk.org> wrote:
>> Magnus Ihse Bursie has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains two commits:
>>
>> - Merge branch 'master' into make-doctor-on-ci
>> - 8345874: Run make doctor automatically on failed CI builds
>
> make/autoconf/spec.gmk.template line 173:
>
>> 171:
>> 172: # dev or ci
>> 173: BUILD_ENV := @BUILD_ENV@
>
> I think the `BUILD_ENV` could be seted in command line also. So, should we update the doc/building.md sync with this change.
What do you mean? This is controlled by the `--with-build-env=[ci|dev]` option. It just needed to be exported from the configure script to the make files.
I am not sure if `--with-build-env` is documented in building.md, but in any case we just list a short selection of options there; the documentation points to `configure --help` to list all currently available options. (It is not feasible to keep the documentation in sync for every possible option.)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24485#discussion_r2063304652
More information about the build-dev
mailing list