Page MenuHomeFreeBSD

graphics/gimp{,-app},graphics/gegl,x11/babl: update to 2.10.38, 0.4.48, 0.1.108
Needs ReviewPublic

Authored by vvd on Sun, May 5, 3:58 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, May 17, 12:01 AM
Unknown Object (File)
Thu, May 16, 3:26 PM
Unknown Object (File)
Tue, May 14, 3:00 PM
Unknown Object (File)
Thu, May 9, 12:19 AM
Unknown Object (File)
Wed, May 8, 7:41 AM
Unknown Object (File)
Wed, May 8, 7:36 AM
Unknown Object (File)
Wed, May 8, 6:13 AM
Unknown Object (File)
Tue, May 7, 8:11 PM
Subscribers

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

vvd requested review of this revision.Sun, May 5, 3:58 PM
vvd created this revision.

All ports should be converted to DISTVERSION as per Porters Handbook

graphics/gegl
Are patch-operations_external_png-load_c and patch-operations_external_tiff-load_c still required?
The OpenMP hack can likely be ripped out by now

x11/babl likely doesn't require libtool, you can probably drop localbase or or at least use localbase:ldflags
SIMD option is only relevant on i386/amd64 (fix) and we can at least always rely on SSE2 on amd64 (requirement).
It would be nice(r) if we could rely on CPUTYPE from the framework

graphics/gimp-app
SIMD option can be retired, we can assume that people what are running gimp aren't using a potato
https://cgit.freebsd.org/ports/tree/graphics/gimp-app/Makefile#n142 - Remove newline
L35 Remove LIB_DEPENDS and replace with \ on the line above
jpeg200 --> JPEG 2000 (https://jpeg.org/jpeg2000/)

All ports should be converted to DISTVERSION as per Porters Handbook

Ok.

graphics/gegl
Are patch-operations_external_png-load_c and patch-operations_external_tiff-load_c still required?

I'll test.

The OpenMP hack can likely be ripped out by now

- #include <altivec.h>
I don't know and I don't have ppc* for testing.
Base 13.3 have this file: /usr/lib/clang/17/include/altivec.h

x11/babl likely doesn't require libtool,

I saw it, but thought the maintainer had kept it for some purpose unknown to me.

you can probably drop localbase or or at least use localbase:ldflags

Will test.

SIMD option is only relevant on i386/amd64 (fix) and we can at least always rely on SSE2 on amd64 (requirement).

Ok, I'll sort out this option.
What is our low mark for i386 - Pentium 2? If yes then we can use MMX as default for i386.

It would be nice(r) if we could rely on CPUTYPE from the framework

Change default depends on CPUTYPE? But if somebody want to turn it off - he can't.

graphics/gimp-app
SIMD option can be retired, we can assume that people what are running gimp aren't using a potato

It's already ON by default.

https://cgit.freebsd.org/ports/tree/graphics/gimp-app/Makefile#n142 - Remove newline

Ok.

L35 Remove LIB_DEPENDS and replace with \ on the line above

Ok.

jpeg200 --> JPEG 2000 (https://jpeg.org/jpeg2000/)

Do you mean replacing the substring jpeg2000 in the description with JPEG 2000?

Thanks for tips!
portclippy printed several warnings too.
I wanted to tidy up these ports with separate commits after updating them.

vvd edited the summary of this revision. (Show Details)
vvd added a subscriber: pkubaj.

Removed patch-operations_external_png-load_c and patch-operations_external_tiff-load_c - build fine without both.

About Altivec:
https://cgit.freebsd.org/ports/commit/graphics/gegl?id=7615d55b290a801f7bee689349a4dc5b3e760acf
https://cgit.freebsd.org/ports/commit/graphics/gegl?id=9105f6edb2964fd253737a70a63cf7c4a146732e
Almost duplicate, but 1st active with clang only and 2nd always. What is better?
Is it build with GCC 12/13/14 and work?
IMHO, we can sort out this after update.

This update also fix this PR: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=278292

Removed graphics/gegl/files/patch-gegl_opencl_cl_platform_h - tested build on powerpc64le (thanks to pkubaj).