On Thu, Apr 26 2018, Marc Espie <[email protected]> wrote:
> On Thu, Apr 26, 2018 at 05:19:39PM +0200, Jeremie Courreges-Anglas wrote:
>> On Wed, Apr 25 2018, Frederic Cambus <[email protected]> wrote:
>> > Hi ports@,
>> >
>> > We now have an up-to-date emulators/mame port for which we can build
>> > and distribute binary packages for amd64 and i386. More information
>> > here: https://marc.info/?l=openbsd-ports&m=151514926004836&w=2
>> >
>> > I think it's time to remove emulators/sdlmame and emulators/sdlmess.
>> > MAME and MESS merged in 2015 so emulators/mame contains both.
>> >
>> > Diff to add @pkgpath markers to emulators/mame and quirks entry below.
>> >
>> > Comments? OK to remove?
>> 
>> Looks like you can't upgrade to mame if you have both sdlmame and
>> sdlmess installed:
>> 
>>   env TRUSTED_PKG_PATH=/usr/ports/packages/amd64/all/ pkg_add -u
>>   [sdlmame-0.160p5]sdlmess-0.160p3->mame-0.197: internal conflict between 
>> mame-0.197 and sdlmame-0.160p5
>> 
>> After removing sdlmame:
>> 
>>   env TRUSTED_PKG_PATH=/usr/ports/packages/amd64/all/ pkg_add -u
>>   sdlmess-0.160p3->mame-0.197: ok
>> 
>> Dunno if that's a limitation in pkg_add or if some annotations/quirks
>> are missing.  Maybe espie knows better?
>> 
>> I'm tempted to ok this anyway...
>> 
> Oh, and does it perform the same if you keep sdlmame, and remove sdlmess
> instead ?

Not directly answering this question because Marc has since pointed me
in the right direction: I had not spotted that sdlmame used subpackages,
so the @pkgpath annotations were slightly off.  Also we ought to provide
a quirks entry for sdlmame-tools, and the obvious REVISION bump for
mame.

Sample pkg_add -u run:

--8<--
# env TRUSTED_PKG_PATH=/tmp/mame/ pkg_add -nu
Ambiguous: choose package for sdlmame-0.160p5
a       0: <None>
        1: mame-0.197p0
        2: sdlmame-0.160p5
Your choice: 0
Ambiguous: choose package for sdlmame-tools-0.160p5
a       0: <None>
        1: mame-0.197p0
        2: sdlmame-tools-0.160p5
Your choice: 0
Ambiguous: choose package for sdlmess-0.160p3
a       0: <None>
        1: mame-0.197p0
        2: sdlmess-0.160p3
Your choice: 0
Couldn't find updates for [snip]
# env TRUSTED_PKG_PATH=/tmp/mame-only/ pkg_add -nu
sdlmame-0.160p5+sdlmame-tools-0.160p5+sdlmess-0.160p3->mame-0.197p0: ok
Couldn't find updates for [snip]
# ls /tmp/mame-only /tmp/mame
/tmp/mame:
mame-0.197p0.tgz            sdlmame-0.160p5.tgz         
sdlmame-tools-0.160p5.tgz   sdlmess-0.160p3.tgz

/tmp/mame-only:
mame-0.197p0.tgz
-->8--

So with the updated diff below along with the removal of sdlmame and
sdlmess, the upgrade path is fully automatic.  naddy is right about the
@pkgpath lines for xmame in sdlmame and sdlmess, but since xmame was
removed in 2010 I don't feel too strongly about it -> not my call.

ok jca@


Index: devel/quirks/Makefile
===================================================================
RCS file: /cvs/ports/devel/quirks/Makefile,v
retrieving revision 1.592
diff -u -p -r1.592 Makefile
--- devel/quirks/Makefile       26 Apr 2018 20:30:23 -0000      1.592
+++ devel/quirks/Makefile       26 Apr 2018 23:43:19 -0000
@@ -5,7 +5,7 @@ CATEGORIES =    devel databases
 DISTFILES =
 
 # API.rev
-PKGNAME =      quirks-2.436
+PKGNAME =      quirks-2.437
 PKG_ARCH =     *
 MAINTAINER =   Marc Espie <[email protected]>
 
Index: devel/quirks/files/Quirks.pm
===================================================================
RCS file: /cvs/ports/devel/quirks/files/Quirks.pm,v
retrieving revision 1.605
diff -u -p -r1.605 Quirks.pm
--- devel/quirks/files/Quirks.pm        26 Apr 2018 20:30:23 -0000      1.605
+++ devel/quirks/files/Quirks.pm        26 Apr 2018 23:43:09 -0000
@@ -526,6 +526,9 @@ my $stem_extensions = {
        'pecl-swish' => 'pecl56-swish',
        'pecl-uploadprogress' => 'pecl56-uploadprogress',
        'rope' => 'py-rope',
+       'sdlmame' => 'mame',
+       'sdlmame-tools' => 'mame',
+       'sdlmess' => 'mame',
 };
 
 my $obsolete_reason = {
Index: emulators/mame/Makefile
===================================================================
RCS file: /cvs/ports/emulators/mame/Makefile,v
retrieving revision 1.8
diff -u -p -r1.8 Makefile
--- emulators/mame/Makefile     25 Apr 2018 10:22:37 -0000      1.8
+++ emulators/mame/Makefile     26 Apr 2018 17:58:53 -0000
@@ -9,6 +9,7 @@ COMMENT =       Multiple Arcade Machine Emulat
 V =            197
 DISTNAME =     mame0${V}s
 PKGNAME =      mame-0.${V}
+REVISION =     0
 
 CATEGORIES =   emulators
 
Index: emulators/mame/pkg/PLIST
===================================================================
RCS file: /cvs/ports/emulators/mame/pkg/PLIST,v
retrieving revision 1.1.1.1
diff -u -p -r1.1.1.1 PLIST
--- emulators/mame/pkg/PLIST    24 Jan 2018 20:56:12 -0000      1.1.1.1
+++ emulators/mame/pkg/PLIST    26 Apr 2018 17:55:56 -0000
@@ -2,6 +2,9 @@
 @conflict sdlmame-*
 @conflict sdlmame-tools-*
 @conflict sdlmess-*
+@pkgpath emulators/sdlmame,-main
+@pkgpath emulators/sdlmame,-tools
+@pkgpath emulators/sdlmess
 @bin bin/castool
 @bin bin/chdman
 @bin bin/floptool


-- 
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF  DDCC 0DFA 74AE 1524 E7EE

Reply via email to