Bug 260215 - java/openjfx14: fix build with non-default CCACHE_DIR
Summary: java/openjfx14: fix build with non-default CCACHE_DIR
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Mikael Urankar
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-12-04 14:09 UTC by Benjamin Takacs
Modified: 2021-12-12 18:51 UTC (History)
1 user (show)

See Also:
bugzilla: maintainer-feedback? (mikael)
nimaje+fbz: merge-quarterly?


Attachments
java/openjfx14: fix build with non-default CCACHE_DIR (1.23 KB, patch)
2021-12-04 14:09 UTC, Benjamin Takacs
nimaje+fbz: maintainer-approval? (mikael)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Benjamin Takacs 2021-12-04 14:09:10 UTC
Created attachment 229893 [details]
java/openjfx14: fix build with non-default CCACHE_DIR

In the build of openjfx14 CCACHE_DIR gets lost leading to build failures e.g. when building as nobody, so add it into ccwrapper and cxxwrapper.

While here remove redundant ${SETENV} in do-build (${_GRADLE_RUN} already contains ${SETENV})

Requesting MFH as this is a build fix.

(I already use this on 12.2 amd64)

QA:
* portlint: No new issues
Comment 1 commit-hook freebsd_committer freebsd_triage 2021-12-12 17:59:36 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=4ddd25225aa38b53f02f4e5b95551cc7cd63647c

commit 4ddd25225aa38b53f02f4e5b95551cc7cd63647c
Author:     Benjamin Takacs <nimaje+fbz@bureaucracy.de>
AuthorDate: 2021-12-12 17:53:29 +0000
Commit:     Mikael Urankar <mikael@FreeBSD.org>
CommitDate: 2021-12-12 17:58:57 +0000

    java/openjfx14: fix build with non-default CCACHE_DIR

    In the build of openjfx14 CCACHE_DIR gets lost leading to build failures e.g.
    when building as nobody, so add it into ccwrapper and cxxwrapper.

    While here remove redundant ${SETENV} in do-build (${_GRADLE_RUN}
    already contains ${SETENV})

    PR:             260215

 java/openjfx14/Makefile | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
Comment 2 commit-hook freebsd_committer freebsd_triage 2021-12-12 18:30:42 UTC
A commit in branch 2021Q4 references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=ce8e04db8b1aa8bf1a3135a38cfc3b4761de0f81

commit ce8e04db8b1aa8bf1a3135a38cfc3b4761de0f81
Author:     Benjamin Takacs <nimaje+fbz@bureaucracy.de>
AuthorDate: 2021-12-12 17:53:29 +0000
Commit:     Mikael Urankar <mikael@FreeBSD.org>
CommitDate: 2021-12-12 18:29:40 +0000

    java/openjfx14: fix build with non-default CCACHE_DIR

    In the build of openjfx14 CCACHE_DIR gets lost leading to build failures e.g.
    when building as nobody, so add it into ccwrapper and cxxwrapper.

    While here remove redundant ${SETENV} in do-build (${_GRADLE_RUN}
    already contains ${SETENV})

    PR:             260215
    (cherry picked from commit 4ddd25225aa38b53f02f4e5b95551cc7cd63647c)

 java/openjfx14/Makefile | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
Comment 3 Mikael Urankar freebsd_committer freebsd_triage 2021-12-12 18:51:25 UTC
Thanks!