Re: mysql_waitpid: broken symbolic link to '/usr/local/bin/mariadb-waitpid'

2020-10-02 Thread Mikolaj Kucharski
On Fri, Oct 02, 2020 at 10:54:05PM +, Mikolaj Kucharski wrote:
> On Thu, Oct 01, 2020 at 09:09:16PM -0400, Brad Smith wrote:
> > 
> > It should go the other way around. Try something like this.
> > 
> 
> I see thanks. Your diff also has the issue which I hit with my diff:
> 
>Can't install mariadb-server-10.5.5p0v1 because of conflicts 
> (mariadb-client-10.5.5p0v1)
> 
> I think it needs to be like sthen@ mentioned:
> 
> @conflict mariadb-server-<10.5.5p0v1
> 
> It's interesting that three people didn't notice that problem, when
> preparing the diff or when reviwing it.
> 
> I'm really curious about rationale of this unexpected behaviour of
> conflict marker. I find it confusing.
> 
> I'm testing Brad's diff with conflict marker fix. Will reply when I'm
> done with final diff.
> 

Below diff works as intended. This is Brad's diff with conflict fix.

Index: Makefile
===
RCS file: /cvs/ports/databases/mariadb/Makefile,v
retrieving revision 1.96
diff -u -p -u -r1.96 Makefile
--- Makefile12 Aug 2020 14:43:54 -  1.96
+++ Makefile3 Oct 2020 00:48:12 -
@@ -9,8 +9,9 @@ DISTNAME=   mariadb-${VERSION}
 PKGNAME-main=  mariadb-client-${VERSION}
 PKGNAME-server=mariadb-server-${VERSION}
 PKGNAME-tests= mariadb-tests-${VERSION}
+REVISION-main= 0
+REVISION-server= 0
 EPOCH= 1
-
 CATEGORIES=databases
 MASTER_SITES=  https://downloads.mariadb.com/MariaDB/${DISTNAME}/source/ \
https://ftp.osuosl.org/pub/mariadb/${DISTNAME}/source/
Index: pkg/PLIST-main
===
RCS file: /cvs/ports/databases/mariadb/pkg/PLIST-main,v
retrieving revision 1.21
diff -u -p -u -r1.21 PLIST-main
--- pkg/PLIST-main  26 Jun 2020 08:46:42 -  1.21
+++ pkg/PLIST-main  3 Oct 2020 00:48:12 -
@@ -1,5 +1,5 @@
 @comment $OpenBSD: PLIST-main,v 1.21 2020/06/26 08:46:42 sthen Exp $
-@conflict mariadb-server-<=10.4.12v1
+@conflict mariadb-server-<10.5.5p0v1
 @conflict mytop-*
 @pkgpath databases/mytop
 @bin bin/mariadb
@@ -18,6 +18,7 @@ bin/mariadb-hotcopy
 bin/mariadb-setpermission
 @bin bin/mariadb-show
 @bin bin/mariadb-slap
+@bin bin/mariadb-waitpid
 @bin bin/mariadb_config
 bin/msql2mysql
 bin/mysql
@@ -88,6 +89,7 @@ lib/pkgconfig/mariadb.pc
 @man man/man1/mariadb-setpermission.1
 @man man/man1/mariadb-show.1
 @man man/man1/mariadb-slap.1
+@man man/man1/mariadb-waitpid.1
 @man man/man1/mariadb.1
 @man man/man1/mariadb_config.1
 @man man/man1/mbstream.1
Index: pkg/PLIST-server
===
RCS file: /cvs/ports/databases/mariadb/pkg/PLIST-server,v
retrieving revision 1.35
diff -u -p -u -r1.35 PLIST-server
--- pkg/PLIST-server12 Aug 2020 14:43:54 -  1.35
+++ pkg/PLIST-server3 Oct 2020 00:48:12 -
@@ -18,7 +18,6 @@ bin/mariadb-install-db
 bin/mariadb-secure-installation
 @bin bin/mariadb-tzinfo-to-sql
 @bin bin/mariadb-upgrade
-@bin bin/mariadb-waitpid
 bin/mariadbd-multi
 bin/mariadbd-safe
 @bin bin/mariadbd-safe-helper
@@ -522,7 +521,6 @@ libexec/mysqld
 @man man/man1/mariadb-secure-installation.1
 @man man/man1/mariadb-tzinfo-to-sql.1
 @man man/man1/mariadb-upgrade.1
-@man man/man1/mariadb-waitpid.1
 @man man/man1/mariadbd-multi.1
 @man man/man1/mariadbd-safe-helper.1
 @man man/man1/mariadbd-safe.1

-- 
Regards,
 Mikolaj



Re: mysql_waitpid: broken symbolic link to '/usr/local/bin/mariadb-waitpid'

2020-10-02 Thread Mikolaj Kucharski
On Thu, Oct 01, 2020 at 09:09:16PM -0400, Brad Smith wrote:
> 
> It should go the other way around. Try something like this.
> 

I see thanks. Your diff also has the issue which I hit with my diff:

   Can't install mariadb-server-10.5.5p0v1 because of conflicts 
(mariadb-client-10.5.5p0v1)

I think it needs to be like sthen@ mentioned:

@conflict mariadb-server-<10.5.5p0v1

It's interesting that three people didn't notice that problem, when
preparing the diff or when reviwing it.

I'm really curious about rationale of this unexpected behaviour of
conflict marker. I find it confusing.

I'm testing Brad's diff with conflict marker fix. Will reply when I'm
done with final diff.


> Index: Makefile
> ===
> RCS file: /home/cvs/ports/databases/mariadb/Makefile,v
> retrieving revision 1.96
> diff -u -p -u -p -r1.96 Makefile
> --- Makefile  12 Aug 2020 14:43:54 -  1.96
> +++ Makefile  30 Sep 2020 20:21:27 -
> @@ -9,8 +9,9 @@ DISTNAME= mariadb-${VERSION}
>  PKGNAME-main=mariadb-client-${VERSION}
>  PKGNAME-server=  mariadb-server-${VERSION}
>  PKGNAME-tests=   mariadb-tests-${VERSION}
> +REVISION-main=   0
> +REVISION-server= 0
>  EPOCH=   1
> -
>  CATEGORIES=  databases
>  MASTER_SITES=
> https://downloads.mariadb.com/MariaDB/${DISTNAME}/source/ \
>   https://ftp.osuosl.org/pub/mariadb/${DISTNAME}/source/
> Index: pkg/PLIST-main
> ===
> RCS file: /home/cvs/ports/databases/mariadb/pkg/PLIST-main,v
> retrieving revision 1.21
> diff -u -p -u -p -r1.21 PLIST-main
> --- pkg/PLIST-main26 Jun 2020 08:46:42 -  1.21
> +++ pkg/PLIST-main30 Sep 2020 21:02:22 -
> @@ -1,5 +1,6 @@
>  @comment $OpenBSD: PLIST-main,v 1.21 2020/06/26 08:46:42 sthen Exp $
>  @conflict mariadb-server-<=10.4.12v1
> +@conflict mariadb-server-<=10.5.5v1
>  @conflict mytop-*
>  @pkgpath databases/mytop
>  @bin bin/mariadb
> @@ -18,6 +19,7 @@ bin/mariadb-hotcopy
>  bin/mariadb-setpermission
>  @bin bin/mariadb-show
>  @bin bin/mariadb-slap
> +@bin bin/mariadb-waitpid
>  @bin bin/mariadb_config
>  bin/msql2mysql
>  bin/mysql
> @@ -88,6 +90,7 @@ lib/pkgconfig/mariadb.pc
>  @man man/man1/mariadb-setpermission.1
>  @man man/man1/mariadb-show.1
>  @man man/man1/mariadb-slap.1
> +@man man/man1/mariadb-waitpid.1
>  @man man/man1/mariadb.1
>  @man man/man1/mariadb_config.1
>  @man man/man1/mbstream.1
> Index: pkg/PLIST-server
> ===
> RCS file: /home/cvs/ports/databases/mariadb/pkg/PLIST-server,v
> retrieving revision 1.35
> diff -u -p -u -p -r1.35 PLIST-server
> --- pkg/PLIST-server  12 Aug 2020 14:43:54 -  1.35
> +++ pkg/PLIST-server  30 Sep 2020 20:19:23 -
> @@ -18,7 +18,6 @@ bin/mariadb-install-db
>  bin/mariadb-secure-installation
>  @bin bin/mariadb-tzinfo-to-sql
>  @bin bin/mariadb-upgrade
> -@bin bin/mariadb-waitpid
>  bin/mariadbd-multi
>  bin/mariadbd-safe
>  @bin bin/mariadbd-safe-helper
> @@ -522,7 +521,6 @@ libexec/mysqld
>  @man man/man1/mariadb-secure-installation.1
>  @man man/man1/mariadb-tzinfo-to-sql.1
>  @man man/man1/mariadb-upgrade.1
> -@man man/man1/mariadb-waitpid.1
>  @man man/man1/mariadbd-multi.1
>  @man man/man1/mariadbd-safe-helper.1
>  @man man/man1/mariadbd-safe.1

-- 
Regards,
 Mikolaj



Re: mysql_waitpid: broken symbolic link to '/usr/local/bin/mariadb-waitpid'

2020-10-01 Thread Brad Smith
On Wed, Sep 30, 2020 at 08:28:53PM +, Mikolaj Kucharski wrote:
> On Mon, Sep 28, 2020 at 02:01:00PM +, Mikolaj Kucharski wrote:
> > Hi,
> > 
> > I've noticed that I have mysql_waitpid installed which is broken
> > symlink to mariadb-waitpid.
> > 
> > I think that symlink should be moved to -server package. I come up with
> > following diff, which I'm still building and didn't test it yet.
> 
> New version after feedback from sthen@
> 
> Something is not right with conflict lines. Is this PEBCAK or
> pkg_add bug?
> 
> This is on a machine without prior mariadb-client, mariadb-server or
> mariadb-tests package installed. Those packages are NOT installed.
> 
> OpenBSD 6.8 (GENERIC.MP) #94: Tue Sep 29 00:13:21 MDT 2020
> dera...@amd64.openbsd.org:/usr/src/sys/arch/amd64/compile/GENERIC.MP
> 
> # env PKG_PATH=https://cdn.openbsd.org/%m \
>   TRUSTED_PKG_PATH=/root/mariadb \
>   pkg_add -Dsnap -i mariadb-client mariadb-server mariadb-tests
> quirks-3.439 signed on 2020-09-30T05:15:47Z
> mariadb-client-10.5.5p0v1: ok
> mariadb-server-10.5.5p0v1:p5-Clone-0.41p0: ok
> mariadb-server-10.5.5p0v1:p5-Math-Base-Convert-0.11p0: ok
> mariadb-server-10.5.5p0v1:p5-Params-Util-1.07p2: ok
> mariadb-server-10.5.5p0v1:p5-Module-Runtime-0.016p0: ok
> mariadb-server-10.5.5p0v1:p5-SQL-Statement-1.412p0: ok
> mariadb-server-10.5.5p0v1:p5-Net-Daemon-0.48p1: ok
> mariadb-server-10.5.5p0v1:p5-PlRPC-0.2020p0: ok
> mariadb-server-10.5.5p0v1:p5-FreezeThaw-0.5001p0: ok
> mariadb-server-10.5.5p0v1:p5-MLDBM-2.05p0: ok
> mariadb-server-10.5.5p0v1:p5-DBI-1.641: ok
> mariadb-server-10.5.5p0v1:p5-DBD-MariaDB-1.21p2: ok
> mariadb-server-10.5.5p0v1:snappy-1.1.8: ok
> Can't install mariadb-server-10.5.5p0v1 because of conflicts 
> (mariadb-client-10.5.5p0v1)
> mariadb-tests-10.5.5v1: ok
> --- mariadb-server-10.5.5p0v1 ---
> Can't install mariadb-server-10.5.5p0v1: conflicts
> Couldn't install mariadb-server-10.5.5p0v1
> 
> # env PKG_PATH=https://cdn.openbsd.org/%m \
>   TRUSTED_PKG_PATH=/root/mariadb \
>   pkg_info -Dsnap -f mariadb-client mariadb-server mariadb-tests | \
>   grep -E '^(Information|@conflict)'
> Information for inst:mariadb-client-10.5.5p0v1
> @conflict mariadb-server-<=10.4.12v1
> @conflict mytop-*
> Information for file:/root/mariadb/mariadb-server-10.5.5p0v1.tgz
> @conflict mariadb-client-<=10.5.5v1
> Information for inst:mariadb-tests-10.5.5v1
> # _
> 
> # ls -lhA /root/mariadb/
> total 168160
> -rw-r--r--  1 root  wheel  10.3M Sep 29 10:35 mariadb-client-10.5.5p0v1.tgz
> -rw-r--r--  1 root  wheel  42.5M Sep 29 10:36 mariadb-server-10.5.5p0v1.tgz
> -rw-r--r--  1 root  wheel  29.2M Sep 29 10:36 mariadb-tests-10.5.5v1.tgz
> 
> 
> Index: Makefile
> ===
> RCS file: /cvs/ports/databases/mariadb/Makefile,v
> retrieving revision 1.96
> diff -u -p -u -r1.96 Makefile
> --- Makefile  12 Aug 2020 14:43:54 -  1.96
> +++ Makefile  30 Sep 2020 20:02:46 -
> @@ -9,6 +9,8 @@ DISTNAME= mariadb-${VERSION}
>  PKGNAME-main=mariadb-client-${VERSION}
>  PKGNAME-server=  mariadb-server-${VERSION}
>  PKGNAME-tests=   mariadb-tests-${VERSION}
> +REVISION-main =  0
> +REVISION-server =0
>  EPOCH=   1
>  
>  CATEGORIES=  databases
> Index: pkg/PLIST-main
> ===
> RCS file: /cvs/ports/databases/mariadb/pkg/PLIST-main,v
> retrieving revision 1.21
> diff -u -p -u -r1.21 PLIST-main
> --- pkg/PLIST-main26 Jun 2020 08:46:42 -  1.21
> +++ pkg/PLIST-main30 Sep 2020 20:02:46 -
> @@ -23,7 +23,6 @@ bin/msql2mysql
>  bin/mysql
>  bin/mysql_config
>  bin/mysql_find_rows
> -bin/mysql_waitpid
>  bin/mysqlaccess
>  bin/mysqladmin
>  bin/mysqlbinlog
> Index: pkg/PLIST-server
> ===
> RCS file: /cvs/ports/databases/mariadb/pkg/PLIST-server,v
> retrieving revision 1.35
> diff -u -p -u -r1.35 PLIST-server
> --- pkg/PLIST-server  12 Aug 2020 14:43:54 -  1.35
> +++ pkg/PLIST-server  30 Sep 2020 20:02:47 -
> @@ -1,5 +1,5 @@
>  @comment $OpenBSD: PLIST-server,v 1.35 2020/08/12 14:43:54 sthen Exp $
> -@conflict mariadb-client-<=10.4.12v1
> +@conflict mariadb-client-<=10.5.5v1
>  @newgroup _mysql:502
>  @newuser _mysql:502:_mysql:daemon:MySQL Account:/nonexistent:/sbin/nologin
>  @rcscript ${RCDIR}/mysqld
> @@ -36,6 +36,7 @@ bin/mysql_secure_installation
>  bin/mysql_setpermission
>  bin/mysql_tzinfo_to_sql
>  bin/mysql_upgrade
> +bin/mysql_waitpid
>  bin/mysqld_multi
>  bin/mysqld_safe
>  bin/mysqld_safe_helper

It should go the other way around. Try something like this.


Index: Makefile
===
RCS file: /home/cvs/ports/databases/mariadb/Makefile,v
retrieving revision 1.96
diff -u -p -u -p -r1.96 Makefile
--- Makefile12 Aug 2020 14:43:54 -  1.96
+++ Makefile30 Sep 2020 20:21:27 

Re: mysql_waitpid: broken symbolic link to '/usr/local/bin/mariadb-waitpid'

2020-10-01 Thread Stuart Henderson
On 2020/09/30 20:28, Mikolaj Kucharski wrote:
> On Mon, Sep 28, 2020 at 02:01:00PM +, Mikolaj Kucharski wrote:
> > Hi,
> > 
> > I've noticed that I have mysql_waitpid installed which is broken
> > symlink to mariadb-waitpid.
> > 
> > I think that symlink should be moved to -server package. I come up with
> > following diff, which I'm still building and didn't test it yet.
> 
> New version after feedback from sthen@
> 
> Something is not right with conflict lines. Is this PEBCAK or
> pkg_add bug?
> 
> This is on a machine without prior mariadb-client, mariadb-server or
> mariadb-tests package installed. Those packages are NOT installed.

Oh, sorry I withdraw my "The rest is correct", thanks for checking :)

>  @comment $OpenBSD: PLIST-server,v 1.35 2020/08/12 14:43:54 sthen Exp $
> -@conflict mariadb-client-<=10.4.12v1
> +@conflict mariadb-client-<=10.5.5v1

It works with

@conflict mariadb-client-<10.5.5p0v1

I think there is some special handling for REVISIONs so that if there's
no vX in the package spec, it matches any of them, I think this is
intentional rather than a bug, but I don't see it described in
packages-specs(7).



Re: mysql_waitpid: broken symbolic link to '/usr/local/bin/mariadb-waitpid'

2020-09-30 Thread Mikolaj Kucharski
On Mon, Sep 28, 2020 at 02:01:00PM +, Mikolaj Kucharski wrote:
> Hi,
> 
> I've noticed that I have mysql_waitpid installed which is broken
> symlink to mariadb-waitpid.
> 
> I think that symlink should be moved to -server package. I come up with
> following diff, which I'm still building and didn't test it yet.

New version after feedback from sthen@

Something is not right with conflict lines. Is this PEBCAK or
pkg_add bug?

This is on a machine without prior mariadb-client, mariadb-server or
mariadb-tests package installed. Those packages are NOT installed.

OpenBSD 6.8 (GENERIC.MP) #94: Tue Sep 29 00:13:21 MDT 2020
dera...@amd64.openbsd.org:/usr/src/sys/arch/amd64/compile/GENERIC.MP

# env PKG_PATH=https://cdn.openbsd.org/%m \
TRUSTED_PKG_PATH=/root/mariadb \
pkg_add -Dsnap -i mariadb-client mariadb-server mariadb-tests
quirks-3.439 signed on 2020-09-30T05:15:47Z
mariadb-client-10.5.5p0v1: ok
mariadb-server-10.5.5p0v1:p5-Clone-0.41p0: ok
mariadb-server-10.5.5p0v1:p5-Math-Base-Convert-0.11p0: ok
mariadb-server-10.5.5p0v1:p5-Params-Util-1.07p2: ok
mariadb-server-10.5.5p0v1:p5-Module-Runtime-0.016p0: ok
mariadb-server-10.5.5p0v1:p5-SQL-Statement-1.412p0: ok
mariadb-server-10.5.5p0v1:p5-Net-Daemon-0.48p1: ok
mariadb-server-10.5.5p0v1:p5-PlRPC-0.2020p0: ok
mariadb-server-10.5.5p0v1:p5-FreezeThaw-0.5001p0: ok
mariadb-server-10.5.5p0v1:p5-MLDBM-2.05p0: ok
mariadb-server-10.5.5p0v1:p5-DBI-1.641: ok
mariadb-server-10.5.5p0v1:p5-DBD-MariaDB-1.21p2: ok
mariadb-server-10.5.5p0v1:snappy-1.1.8: ok
Can't install mariadb-server-10.5.5p0v1 because of conflicts 
(mariadb-client-10.5.5p0v1)
mariadb-tests-10.5.5v1: ok
--- mariadb-server-10.5.5p0v1 ---
Can't install mariadb-server-10.5.5p0v1: conflicts
Couldn't install mariadb-server-10.5.5p0v1

# env PKG_PATH=https://cdn.openbsd.org/%m \
TRUSTED_PKG_PATH=/root/mariadb \
pkg_info -Dsnap -f mariadb-client mariadb-server mariadb-tests | \
grep -E '^(Information|@conflict)'
Information for inst:mariadb-client-10.5.5p0v1
@conflict mariadb-server-<=10.4.12v1
@conflict mytop-*
Information for file:/root/mariadb/mariadb-server-10.5.5p0v1.tgz
@conflict mariadb-client-<=10.5.5v1
Information for inst:mariadb-tests-10.5.5v1
# _

# ls -lhA /root/mariadb/
total 168160
-rw-r--r--  1 root  wheel  10.3M Sep 29 10:35 mariadb-client-10.5.5p0v1.tgz
-rw-r--r--  1 root  wheel  42.5M Sep 29 10:36 mariadb-server-10.5.5p0v1.tgz
-rw-r--r--  1 root  wheel  29.2M Sep 29 10:36 mariadb-tests-10.5.5v1.tgz


Index: Makefile
===
RCS file: /cvs/ports/databases/mariadb/Makefile,v
retrieving revision 1.96
diff -u -p -u -r1.96 Makefile
--- Makefile12 Aug 2020 14:43:54 -  1.96
+++ Makefile30 Sep 2020 20:02:46 -
@@ -9,6 +9,8 @@ DISTNAME=   mariadb-${VERSION}
 PKGNAME-main=  mariadb-client-${VERSION}
 PKGNAME-server=mariadb-server-${VERSION}
 PKGNAME-tests= mariadb-tests-${VERSION}
+REVISION-main =0
+REVISION-server =  0
 EPOCH= 1
 
 CATEGORIES=databases
Index: pkg/PLIST-main
===
RCS file: /cvs/ports/databases/mariadb/pkg/PLIST-main,v
retrieving revision 1.21
diff -u -p -u -r1.21 PLIST-main
--- pkg/PLIST-main  26 Jun 2020 08:46:42 -  1.21
+++ pkg/PLIST-main  30 Sep 2020 20:02:46 -
@@ -23,7 +23,6 @@ bin/msql2mysql
 bin/mysql
 bin/mysql_config
 bin/mysql_find_rows
-bin/mysql_waitpid
 bin/mysqlaccess
 bin/mysqladmin
 bin/mysqlbinlog
Index: pkg/PLIST-server
===
RCS file: /cvs/ports/databases/mariadb/pkg/PLIST-server,v
retrieving revision 1.35
diff -u -p -u -r1.35 PLIST-server
--- pkg/PLIST-server12 Aug 2020 14:43:54 -  1.35
+++ pkg/PLIST-server30 Sep 2020 20:02:47 -
@@ -1,5 +1,5 @@
 @comment $OpenBSD: PLIST-server,v 1.35 2020/08/12 14:43:54 sthen Exp $
-@conflict mariadb-client-<=10.4.12v1
+@conflict mariadb-client-<=10.5.5v1
 @newgroup _mysql:502
 @newuser _mysql:502:_mysql:daemon:MySQL Account:/nonexistent:/sbin/nologin
 @rcscript ${RCDIR}/mysqld
@@ -36,6 +36,7 @@ bin/mysql_secure_installation
 bin/mysql_setpermission
 bin/mysql_tzinfo_to_sql
 bin/mysql_upgrade
+bin/mysql_waitpid
 bin/mysqld_multi
 bin/mysqld_safe
 bin/mysqld_safe_helper

-- 
Regards,
 Mikolaj



Re: mysql_waitpid: broken symbolic link to '/usr/local/bin/mariadb-waitpid'

2020-09-28 Thread Mikolaj Kucharski
On Mon, Sep 28, 2020 at 08:09:28PM +0100, Stuart Henderson wrote:
> On 2020/09/28 19:01, Mikolaj Kucharski wrote:
> > On Mon, Sep 28, 2020 at 07:54:01PM +0100, Stuart Henderson wrote:
> > > > --- pkg/PLIST-main  26 Jun 2020 08:46:42 -  1.21
> > > > +++ pkg/PLIST-main  28 Sep 2020 13:57:54 -
> > > > @@ -1,5 +1,5 @@
> > > >  @comment $OpenBSD: PLIST-main,v 1.21 2020/06/26 08:46:42 sthen Exp $
> > > > -@conflict mariadb-server-<=10.4.12v1
> > > > +@conflict mariadb-server-<=10.5.5v1
> > > 
> > > This line does not need to change
> > > 
> > 
> > Are you sure? I didn't had a chance to test various install or upgrade
> > scenarios, but conflict is there because file exists in old -main and
> > new -server package, so I'm tryig to avoid situation when old -main and
> > new -server will somehow end up installed and fail.
> > 
> > Dunno, maybe I'm missing something in my mental picture.
> 
> You already avoided that situation with the @conflict marker in the new
> -server version.
> 

Ah, thank you!

-- 
Regards,
 Mikolaj



Re: mysql_waitpid: broken symbolic link to '/usr/local/bin/mariadb-waitpid'

2020-09-28 Thread Stuart Henderson
On 2020/09/28 19:01, Mikolaj Kucharski wrote:
> On Mon, Sep 28, 2020 at 07:54:01PM +0100, Stuart Henderson wrote:
> > > --- pkg/PLIST-main26 Jun 2020 08:46:42 -  1.21
> > > +++ pkg/PLIST-main28 Sep 2020 13:57:54 -
> > > @@ -1,5 +1,5 @@
> > >  @comment $OpenBSD: PLIST-main,v 1.21 2020/06/26 08:46:42 sthen Exp $
> > > -@conflict mariadb-server-<=10.4.12v1
> > > +@conflict mariadb-server-<=10.5.5v1
> > 
> > This line does not need to change
> > 
> 
> Are you sure? I didn't had a chance to test various install or upgrade
> scenarios, but conflict is there because file exists in old -main and
> new -server package, so I'm tryig to avoid situation when old -main and
> new -server will somehow end up installed and fail.
> 
> Dunno, maybe I'm missing something in my mental picture.

You already avoided that situation with the @conflict marker in the new
-server version.



Re: mysql_waitpid: broken symbolic link to '/usr/local/bin/mariadb-waitpid'

2020-09-28 Thread Mikolaj Kucharski
On Mon, Sep 28, 2020 at 07:54:01PM +0100, Stuart Henderson wrote:
> > --- pkg/PLIST-main  26 Jun 2020 08:46:42 -  1.21
> > +++ pkg/PLIST-main  28 Sep 2020 13:57:54 -
> > @@ -1,5 +1,5 @@
> >  @comment $OpenBSD: PLIST-main,v 1.21 2020/06/26 08:46:42 sthen Exp $
> > -@conflict mariadb-server-<=10.4.12v1
> > +@conflict mariadb-server-<=10.5.5v1
> 
> This line does not need to change
> 

Are you sure? I didn't had a chance to test various install or upgrade
scenarios, but conflict is there because file exists in old -main and
new -server package, so I'm tryig to avoid situation when old -main and
new -server will somehow end up installed and fail.

Dunno, maybe I'm missing something in my mental picture.

-- 
Regards,
 Mikolaj



Re: mysql_waitpid: broken symbolic link to '/usr/local/bin/mariadb-waitpid'

2020-09-28 Thread Stuart Henderson
On 2020/09/28 14:01, Mikolaj Kucharski wrote:
> Hi,
> 
> I've noticed that I have mysql_waitpid installed which is broken
> symlink to mariadb-waitpid.
> 
> I think that symlink should be moved to -server package. I come up with
> following diff, which I'm still building and didn't test it yet.
> 
> 
> Index: Makefile
> ===
> RCS file: /cvs/ports/databases/mariadb/Makefile,v
> retrieving revision 1.96
> diff -u -p -u -r1.96 Makefile
> --- Makefile  12 Aug 2020 14:43:54 -  1.96
> +++ Makefile  28 Sep 2020 13:57:54 -
> @@ -9,6 +9,8 @@ DISTNAME= mariadb-${VERSION}
>  PKGNAME-main=mariadb-client-${VERSION}
>  PKGNAME-server=  mariadb-server-${VERSION}
>  PKGNAME-tests=   mariadb-tests-${VERSION}
> +REVISION-main =  0
> +REVISION-server =0
>  EPOCH=   1
>  
>  CATEGORIES=  databases
> Index: pkg/PLIST-main
> ===
> RCS file: /cvs/ports/databases/mariadb/pkg/PLIST-main,v
> retrieving revision 1.21
> diff -u -p -u -r1.21 PLIST-main
> --- pkg/PLIST-main26 Jun 2020 08:46:42 -  1.21
> +++ pkg/PLIST-main28 Sep 2020 13:57:54 -
> @@ -1,5 +1,5 @@
>  @comment $OpenBSD: PLIST-main,v 1.21 2020/06/26 08:46:42 sthen Exp $
> -@conflict mariadb-server-<=10.4.12v1
> +@conflict mariadb-server-<=10.5.5v1

This line does not need to change

>  @conflict mytop-*
>  @pkgpath databases/mytop
>  @bin bin/mariadb
> @@ -23,7 +23,6 @@ bin/msql2mysql
>  bin/mysql
>  bin/mysql_config
>  bin/mysql_find_rows
> -bin/mysql_waitpid
>  bin/mysqlaccess
>  bin/mysqladmin
>  bin/mysqlbinlog
> Index: pkg/PLIST-server
> ===
> RCS file: /cvs/ports/databases/mariadb/pkg/PLIST-server,v
> retrieving revision 1.35
> diff -u -p -u -r1.35 PLIST-server
> --- pkg/PLIST-server  12 Aug 2020 14:43:54 -  1.35
> +++ pkg/PLIST-server  28 Sep 2020 13:57:54 -
> @@ -1,5 +1,5 @@
>  @comment $OpenBSD: PLIST-server,v 1.35 2020/08/12 14:43:54 sthen Exp $
> -@conflict mariadb-client-<=10.4.12v1
> +@conflict mariadb-client-<=10.5.5v1
>  @newgroup _mysql:502
>  @newuser _mysql:502:_mysql:daemon:MySQL Account:/nonexistent:/sbin/nologin
>  @rcscript ${RCDIR}/mysqld
> @@ -36,6 +36,7 @@ bin/mysql_secure_installation
>  bin/mysql_setpermission
>  bin/mysql_tzinfo_to_sql
>  bin/mysql_upgrade
> +bin/mysql_waitpid
>  bin/mysqld_multi
>  bin/mysqld_safe
>  bin/mysqld_safe_helper

The rest is correct and is OK sthen@



mysql_waitpid: broken symbolic link to '/usr/local/bin/mariadb-waitpid'

2020-09-28 Thread Mikolaj Kucharski
Hi,

I've noticed that I have mysql_waitpid installed which is broken
symlink to mariadb-waitpid.

I think that symlink should be moved to -server package. I come up with
following diff, which I'm still building and didn't test it yet.


Index: Makefile
===
RCS file: /cvs/ports/databases/mariadb/Makefile,v
retrieving revision 1.96
diff -u -p -u -r1.96 Makefile
--- Makefile12 Aug 2020 14:43:54 -  1.96
+++ Makefile28 Sep 2020 13:57:54 -
@@ -9,6 +9,8 @@ DISTNAME=   mariadb-${VERSION}
 PKGNAME-main=  mariadb-client-${VERSION}
 PKGNAME-server=mariadb-server-${VERSION}
 PKGNAME-tests= mariadb-tests-${VERSION}
+REVISION-main =0
+REVISION-server =  0
 EPOCH= 1
 
 CATEGORIES=databases
Index: pkg/PLIST-main
===
RCS file: /cvs/ports/databases/mariadb/pkg/PLIST-main,v
retrieving revision 1.21
diff -u -p -u -r1.21 PLIST-main
--- pkg/PLIST-main  26 Jun 2020 08:46:42 -  1.21
+++ pkg/PLIST-main  28 Sep 2020 13:57:54 -
@@ -1,5 +1,5 @@
 @comment $OpenBSD: PLIST-main,v 1.21 2020/06/26 08:46:42 sthen Exp $
-@conflict mariadb-server-<=10.4.12v1
+@conflict mariadb-server-<=10.5.5v1
 @conflict mytop-*
 @pkgpath databases/mytop
 @bin bin/mariadb
@@ -23,7 +23,6 @@ bin/msql2mysql
 bin/mysql
 bin/mysql_config
 bin/mysql_find_rows
-bin/mysql_waitpid
 bin/mysqlaccess
 bin/mysqladmin
 bin/mysqlbinlog
Index: pkg/PLIST-server
===
RCS file: /cvs/ports/databases/mariadb/pkg/PLIST-server,v
retrieving revision 1.35
diff -u -p -u -r1.35 PLIST-server
--- pkg/PLIST-server12 Aug 2020 14:43:54 -  1.35
+++ pkg/PLIST-server28 Sep 2020 13:57:54 -
@@ -1,5 +1,5 @@
 @comment $OpenBSD: PLIST-server,v 1.35 2020/08/12 14:43:54 sthen Exp $
-@conflict mariadb-client-<=10.4.12v1
+@conflict mariadb-client-<=10.5.5v1
 @newgroup _mysql:502
 @newuser _mysql:502:_mysql:daemon:MySQL Account:/nonexistent:/sbin/nologin
 @rcscript ${RCDIR}/mysqld
@@ -36,6 +36,7 @@ bin/mysql_secure_installation
 bin/mysql_setpermission
 bin/mysql_tzinfo_to_sql
 bin/mysql_upgrade
+bin/mysql_waitpid
 bin/mysqld_multi
 bin/mysqld_safe
 bin/mysqld_safe_helper

-- 
Regards,
 Mikolaj