Bug 281868 - graphics/embree: fix build with clang 19
Summary: graphics/embree: fix build with clang 19
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Alexey Dokuchaev
URL:
Keywords:
Depends on:
Blocks: 280562
  Show dependency treegraph
 
Reported: 2024-10-05 14:26 UTC by Dimitry Andric
Modified: 2024-10-05 16:37 UTC (History)
0 users

See Also:
bugzilla: maintainer-feedback? (danfe)


Attachments
graphics/embree: fix build with clang 19 (2.43 KB, patch)
2024-10-05 14:27 UTC, Dimitry Andric
danfe: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dimitry Andric freebsd_committer freebsd_triage 2024-10-05 14:26:52 UTC
Clang 19 has become more strict about errors in member functions, which
results in errors building graphics/embree:

  /wrkdirs/usr/ports/graphics/embree/work/embree-3.13.5/kernels/geometry/../common/../subdiv/bezier_curve.h:138:56: error: no member named 'u' in 'QuadraticBezierCurve<V>'
    138 |         return cout << "QuadraticBezierCurve ( (" << a.u.lower << ", " << a.u.upper << "), " << a.v0 << ", " << a.v1 << ", " << a.v2 << ")";
        |                                                      ~ ^
  /wrkdirs/usr/ports/graphics/embree/work/embree-3.13.5/kernels/geometry/../common/../subdiv/bezier_curve.h:138:77: error: no member named 'u' in 'QuadraticBezierCurve<V>'
    138 |         return cout << "QuadraticBezierCurve ( (" << a.u.lower << ", " << a.u.upper << "), " << a.v0 << ", " << a.v1 << ", " << a.v2 << ")";
        |                                                                           ~ ^

Upstream fixed this for https://github.com/RenderKit/embree/issues/486
in https://github.com/RenderKit/embree/commit/cda4cf191, so add a
PATCHFILES entry for it.
Comment 1 Dimitry Andric freebsd_committer freebsd_triage 2024-10-05 14:27:40 UTC
Created attachment 254019 [details]
graphics/embree: fix build with clang 19
Comment 2 Alexey Dokuchaev freebsd_committer freebsd_triage 2024-10-05 16:20:54 UTC
Comment on attachment 254019 [details]
graphics/embree: fix build with clang 19

Looks good, as always.  Feel free to PATCHFILES-fix any of my other ports when it suffices, without opening a PR, thank you!
Comment 3 commit-hook freebsd_committer freebsd_triage 2024-10-05 16:34:26 UTC
A commit in branch main references this bug:

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

commit 7bf7810c8515f9008d6c86ca88a9838aeb886942
Author:     Dimitry Andric <dim@FreeBSD.org>
AuthorDate: 2024-10-05 14:27:05 +0000
Commit:     Dimitry Andric <dim@FreeBSD.org>
CommitDate: 2024-10-05 16:33:25 +0000

    graphics/embree: fix build with clang 19

    Clang 19 has become more strict about errors in member functions, which
    results in errors building graphics/embree:

      /wrkdirs/usr/ports/graphics/embree/work/embree-3.13.5/kernels/geometry/../common/../subdiv/bezier_curve.h:138:56: error: no member named 'u' in 'QuadraticBezierCurve<V>'
        138 |         return cout << "QuadraticBezierCurve ( (" << a.u.lower << ", " << a.u.upper << "), " << a.v0 << ", " << a.v1 << ", " << a.v2 << ")";
            |                                                      ~ ^
      /wrkdirs/usr/ports/graphics/embree/work/embree-3.13.5/kernels/geometry/../common/../subdiv/bezier_curve.h:138:77: error: no member named 'u' in 'QuadraticBezierCurve<V>'
        138 |         return cout << "QuadraticBezierCurve ( (" << a.u.lower << ", " << a.u.upper << "), " << a.v0 << ", " << a.v1 << ", " << a.v2 << ")";
            |                                                                           ~ ^

    Upstream fixed this for https://github.com/RenderKit/embree/issues/486
    in https://github.com/RenderKit/embree/commit/cda4cf191, so add a
    PATCHFILES entry for it.

    PR:             281868
    Approved by:    danfe (maintainer)
    MFH:            2024Q3

 graphics/embree/Makefile | 3 +++
 graphics/embree/distinfo | 4 +++-
 2 files changed, 6 insertions(+), 1 deletion(-)
Comment 4 commit-hook freebsd_committer freebsd_triage 2024-10-05 16:36:27 UTC
A commit in branch 2024Q3 references this bug:

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

commit 787b4f9530327fab513eaf8810bd63ace9172476
Author:     Dimitry Andric <dim@FreeBSD.org>
AuthorDate: 2024-10-05 14:27:05 +0000
Commit:     Dimitry Andric <dim@FreeBSD.org>
CommitDate: 2024-10-05 16:35:25 +0000

    graphics/embree: fix build with clang 19

    Clang 19 has become more strict about errors in member functions, which
    results in errors building graphics/embree:

      /wrkdirs/usr/ports/graphics/embree/work/embree-3.13.5/kernels/geometry/../common/../subdiv/bezier_curve.h:138:56: error: no member named 'u' in 'QuadraticBezierCurve<V>'
        138 |         return cout << "QuadraticBezierCurve ( (" << a.u.lower << ", " << a.u.upper << "), " << a.v0 << ", " << a.v1 << ", " << a.v2 << ")";
            |                                                      ~ ^
      /wrkdirs/usr/ports/graphics/embree/work/embree-3.13.5/kernels/geometry/../common/../subdiv/bezier_curve.h:138:77: error: no member named 'u' in 'QuadraticBezierCurve<V>'
        138 |         return cout << "QuadraticBezierCurve ( (" << a.u.lower << ", " << a.u.upper << "), " << a.v0 << ", " << a.v1 << ", " << a.v2 << ")";
            |                                                                           ~ ^

    Upstream fixed this for https://github.com/RenderKit/embree/issues/486
    in https://github.com/RenderKit/embree/commit/cda4cf191, so add a
    PATCHFILES entry for it.

    PR:             281868
    Approved by:    danfe (maintainer)
    MFH:            2024Q3

    (cherry picked from commit 7bf7810c8515f9008d6c86ca88a9838aeb886942)

 graphics/embree/Makefile | 3 +++
 graphics/embree/distinfo | 4 +++-
 2 files changed, 6 insertions(+), 1 deletion(-)