Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 9 Jan 2017 11:22:04 -0600
From:      John Marino <freebsd.contact@marino.st>
To:        Adam Weinberger <adamw@FreeBSD.org>, ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   Re: svn commit: r430981 - head/editors/vim
Message-ID:  <50df5703-a6da-e25f-5f48-6501dbc80ed0@marino.st>
In-Reply-To: <201701091703.v09H3cEE082911@repo.freebsd.org>
References:  <201701091703.v09H3cEE082911@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On 1/9/2017 11:03, Adam Weinberger wrote:
> Author: adamw
> Date: Mon Jan  9 17:03:37 2017
> New Revision: 430981
> URL: https://svnweb.freebsd.org/changeset/ports/430981
>
> Log:
>   Re-add MAKE_JOBS_UNSAFE. From mat:
>     --- scratch ---
>     cp config.mk.dist auto/config.mk
>     --- clean ---
>     make[2]: "/wrkdirs/usr/ports/editors/vim/work/vim-8.0.0149/src/po/Makefile" line 4: Could not find ../auto/config.mk
>     make[2]: Fatal errors encountered -- cannot continue
>
>   Install desktop files and icons when the GNOME, GTK2, or GTK3 knobs are turned
>   on. Requested by Kevin Zheng. PORTREVISION bump for this.
>

Hi Adam,
So I looked up the commit history since this message made me curious and 
this comes from 10 DEC 16:
"Patch 129 was a fix for parallel make. It builds fine for me on
FreeBSD with -j4, and on macOS with -j8, but that's the extent
of what I can test on my own. I'm removing MAKE_JOBS_UNSAFE with
this commit, but if one of you with your crazy 256-core machines
encounters build failures then please let me know!"

I've seen this kind of thing from time-to-time, where somebody like me, 
after being 100% sure and, marks a port jobs unsafe usually documenting 
why.  Then later, somebody tries to recreate it on some random machine, 
can't do it, and decides, "Hey, it must have magically fixed itself" and 
removes the label.  And then, of course, it's actually still broken and 
the original committer often has to relabel the port unsafe.

What should happen is that the original cause for jobs unsafety has to 
be traced, and then either A) patched to fix to B) confirm concretely 
that upstream has identified and fixed the problem.  Without concrete 
proof that a port has been fixed, IMO it should remain unsafe indefinitely.

I don't think this is written down anywhere, but it would be nice if it 
were documented in a guide, perhaps the do's/dont's for ports committers 
because incorrect reversion of MAKE_JOBS_UNSAFE happens much more often 
than it should.

No, it's not the worst thing in the world, but I think as a group we can 
do better in this area.  Reproducing jobs unsafety is not always easy, 
nor is it a simple matter of -j number.

John





---
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?50df5703-a6da-e25f-5f48-6501dbc80ed0>