On 2021/07/29 13:50, Marc Espie wrote:
> Well, a lot of porters, myself included, have aliases and scripts to edit
> files that make use of the .orig suffix, so there will be a lot of individual
> churn in the second case.
> 
> I expect rust to be a Special Needs language, so I'm not sure the second
> solution is worth it.

Some of these are rust, but most are not

cad/netgen/Makefile:27:PATCHORIG =      .patch.orig
devel/angr/vex/Makefile:17:PATCHORIG =  .pat.orig
devel/cbindgen/Makefile:77:PATCHORIG =  .ports.orig
devel/electron/Makefile:46:PATCHORIG=   .orig.port
devel/git/Makefile:102:PATCHORIG =              .pat.orig
devel/rats/Makefile:21:PATCHORIG=               .orig.port
editors/libreoffice/Makefile:277:PATCHORIG=     .orig.port
editors/scintilla/Makefile:21:PATCHORIG=        .orig.port
games/0ad/base/Makefile:55:PATCHORIG =          .orig.port
games/libgdx/Makefile:44:PATCHORIG =            .orig.port
games/vegastrike/engine/Makefile:20:PATCHORIG =         .orig.port
lang/rust/Makefile:173:PATCHORIG =              .openbsd.orig
mail/mlmmj/Makefile:22:PATCHORIG=       .pat.orig
math/graphviz/Makefile:15:PATCHORIG=    .orig2
net/bore/Makefile:18:PATCHORIG =                .openbsd.orig
net/isc-dhcp/Makefile:17:PATCHORIG=     .orig.port
print/ghostscript/gnu/Makefile:139:PATCHORIG=   .pat.orig
security/fwbuilder/Makefile:27:PATCHORIG =      .orig.port
security/hydra/Makefile:40:PATCHORIG=   .orig.port
security/py-passlib/Makefile:22:PATCHORIG=      .orig.port
security/sn0int/Makefile:38:PATCHORIG =         .openbsd.orig
sysutils/bacula/Makefile:50:PATCHORIG=          .orig.port
sysutils/flashrom/Makefile:29:PATCHORIG=        .orig.port
textproc/latex2html/Makefile:16:PATCHORIG=      .pat.orig
textproc/py-sphinx-notfound-page/Makefile:18:PATCHORIG =                
.orig.port
textproc/ripgrep/Makefile:89:PATCHORIG =                .openbsd.orig
textproc/tree-sitter/Makefile:123:PATCHORIG =           .openbsd.orig
www/chromium/Makefile:51:PATCHORIG=     .orig.port
www/httrack/Makefile:50:PATCHORIG =             .ports.orig
www/iridium/Makefile:52:PATCHORIG=      .orig.port
www/nextcloud_notify_push/Makefile:26:PATCHORIG =               .openbsd.orig
www/tor-browser/browser/Makefile:16:PATCHORIG =         .pat.orig
www/zola/Makefile:329:PATCHORIG =               .openbsd.orig
x11/cegui/Makefile:9:PATCHORIG =                .orig.port
x11/qt5/qtwebengine/Makefile:103:PATCHORIG=             .orig.port
x11/tigervnc/Makefile:55:PATCHORIG=     .orig.port

This is what I have now to deal with it, which works OK if I can run
it from the /usr/ports directory with a full path, but a lot of my real
world editing is done after "make patch" via "find . -name rej" in
wrksrc so the suffix isn't available easily. So a one-time change
that avoids the special cases would be quite welcome to me ..

function vibak { 
        local _orig=.orig
        pwd|grep -q ^/usr/ports && [ -r distinfo ] && _orig=`make 
show=PATCHORIG`
        for i; do [ -f "$i$_orig" ] || doas -u _pbuild cp "$i" "$i$_orig"; 
done; doas -u _pbuild env HOME=/home/pbuild $EDITOR "$@" ; }

Reply via email to