Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 21 Mar 2017 18:43:01 +0100 (CET)
From:      Gerald Pfeifer <gerald@pfeifer.com>
To:        Gleb Smirnoff <glebius@FreeBSD.org>
Cc:        ports-committers@freebsd.org, svn-ports-all@freebsd.org,  svn-ports-head@freebsd.org
Subject:   Re: svn commit: r436576 - in head/emulators/wine-devel: . files
Message-ID:  <alpine.LNX.2.21.1703211835270.28312@anthias.pfeifer.com>
In-Reply-To: <201703210858.v2L8w0Zr014870@repo.freebsd.org>
References:  <201703210858.v2L8w0Zr014870@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Hi Gleb,

On Tue, 21 Mar 2017, Gleb Smirnoff wrote:
> Log:
>   Make it compilable on FreeBSD 12 after struct inpcb and struct tcpcb 
>   were stopped being exported.
> Added:
>   head/emulators/wine-devel/files/patch-dlls_iphlpapi_ipstats.c   (contents, props changed)

thanks for creating a fix for this and sharing it.

> +PORTREVISION=	1

This actually isn't necessary and triggers a needless rebuild for
users on older versions of FreeBSD than that.

(When the build is broken under some circumstances, but not others, 
simply fixing it is sufficient.)

This is why (gentle hint) it actually does make sense to working 
with the maintainer of a port instead of just committing a fix. :-)

> Added: head/emulators/wine-devel/files/patch-dlls_iphlpapi_ipstats.c
> ==============================================================================

Have you submitted this upstream as well?  We should minimize local
patches like this, not the least since some people (and my nightly
automated tester, for example) also build from upstream sources.

And it adds maintenance overhead over time.


If you have not submitted it yet, 
  https://wiki.winehq.org/Submitting_Patches
has good information (if a bit much I'll admit) and I'll be happy
to help.

At a minimum, if you don't want to follow the regular process, 
sending it to wine-patches@winehq.com copying me would be good.

Thanks,
Gerald



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?alpine.LNX.2.21.1703211835270.28312>