On Tue, 28 Jan 2025, s...@triumf.ca wrote:
> FWIW, after a bit of effort, I managed to make wine compile and run
> without the gcc dependency (which, when installed, can mess other
> ports up during/after update).
> 
> See attached files. USB not tested yet.

Ah, so GCC only as a build time dependency?

Did you try the USE_GCC=yes:build syntax? If that does not suffice (which 
I can see may be the case reviewing at Mk/bsd.gcc.mk and your update) look 
into what it might take to address it there to also benefit others with a 
similar need and avoid some of the manual coding of your approach?


Note, at this point I am not planning to make this change or a variation 
thereof since it's not what the majority of other system and upstream do 
(to the best of my knowledge) and Wine on FreeBSD is tricky enough as it 
is.

Note 2, I stepped down from maintaining wine-devel, though I did keep it 
up to date and bring it into the Wine 10.x era while also updating the 
main wine port to Wine 10.0. Should a new maintainer step up, which I 
hope, maybe they'll make this change. A drive by commit with bigger 
changes like this without assuming maintainership would not be great,
though.


Also, you made a number of further changes compared to current wine-devel:
 · reductions to CONFIGURE_ARGS, which qI'm not sure why,
 · bumping PORTEPOCH (which looks very wrong), and 
 · dropping the use of gstreamer.

On the latter there is quite some history here from

  commit 27a252c686ad65fff1df4efe19209385408e1f81
  Author: Gerald Pfeifer <ger...@freebsd.org>
  Date:   Sat Aug 27 20:56:47 2011 +0000

    Update to Wine 1.3.26.  This includes the following changes:
     - Support for multisampling in Direct3D.
     - New version of the Gecko engine.
     - Improvements to the network proxy handling.
     - Better write support in MSXML.
     - Side-by-side manifests for built-in libraries.
     - Various bug fixes.
    
    Explicitly disable gstreamer support, thus avoiding automatic
    detection and incorrect package lists in case. [1]
    
    PR:             159749 [1]
    Submitted by:   Anatoly Borodin <anatoly.boro...@gmail.com> [1]

over enabling it

  commit 63ef7d1ab018109313476c9b03632c4fe1ab6633
  Author: Gerald Pfeifer <ger...@freebsd.org>
  Date:   Sun Oct 22 16:07:12 2023 +0000

    emulators/wine-devel: Use GStreamer when supporting sound
    
    When ALSA or OSS are enabled, i.e., we have sound enabled, pull
    in and use GStreamer as well.
    
    PR:             273987
    Reported by:    scf

to refining it

  commit 132921c8f898c5ca11780b9b87543cce6eb406d2
  Author: Alastair Hogge <a...@riseup.net>
  Date:   Thu Nov 16 11:41:07 2023 +0000

    emulators/wine-devel: GSTREAMER dependency cleanup

    * Add missing USES=gnome and USE=GNOME=glib20 for GSTREAMER dependencies
    * Add ${STRIP_CMD} to GNU configure environment variable ac_cv_prog_STRIP

    PR:             274953

This does not mean your change there is intrinsically bad or undesirable. 
Just be explicit about it and its rationale and listen to what Sean and 
Alistair have to say (regarding their changes and current experiences).

Cheers,
Gerald

PS: In general, my strong recommendation is to make one change at a time, 
sharing, discussing and validating it with those interested, instead of 
bundling.

Reply via email to