databases/automysqlbackup: add initial version of port (3.0_rc6)

[me_cschwarz.com]

Summary

port databases/autobackupmysql exists, but does not have the same featureset

Test Plan

Works in my MySQL jail (mariadb-103-{server,client})

[matthew]

databases/automysqlbackup/Makefile

29

${MKDIR} already includes the '-p' flag: no need to repeat it.

databases/automysqlbackup/pkg-plist

4

You aren't recreating the 0700 directory permissions that you set in the makefile -- this will end up wit 0755 permissionns if you install from pkg.

[me_cschwarz.com]

address first review round remarks
(where are the docs for @dir params?)

[me_cschwarz.com]

databases/automysqlbackup/pkg-plist

4

where is the documentation for @dir?

[mmokhi]

databases/automysqlbackup/Makefile

30

I'm not sure if I understood @matthew 's feedback on this line...
But I would personally let the chmod stuff being done in pkg-plist instead of Makefile... (Or at least either all here or all there)...

databases/automysqlbackup/files/patch-automysqlbackup

10

Maybe it's too much what I'm pointing, but I often use %LOCALBASE% (or in this case %ETCDIR%) and the ${REINPLACE_CMD} in post-patch 😅
Not saying it's better or worse to do so, but might help other people (who have diff place that /usr/local to use it as well 😊

databases/automysqlbackup/pkg-plist

4

This is probably what you are looking for. 👍

[matthew]

databases/automysqlbackup/Makefile

30

When you are building a port, all the Makefile installation commands are doing is copying files into ${STAGEDIR}. The contents of ${STAGEDIR} are then turned into a pkg -- either gathered into a pkg tarball, or in it's installed form with files spread across your active filesytem. This happens whether or not you install directly from your ports tree, or whether you create a pkg tarball, stick it into a repository and download and install from there.

You're meant to be able to install to ${STAGEDIR} and thence build a pkg as an unprivileged user, which means that file ownership and permissions in ${STAGEDIR} can't just be copied into the pkg generated from ${STAGEDIR}, and thence into the main filesystem when the pkg is installed -- in fact, they are ignored, other than copying over any execute permissions. In particular anything like setuid or setgid bits are stripped; regular files default to root:wheel and mode 0644 or 0755 if executable, and directories to root:wheel and mode 0755. If you need any other permissions or ownership, this needs to be set in the pkg-plist.

In summary: permissions and ownership in ${STAGEDIR} are (almost) irrelevant: it's what's set in pkg-plist that counts.

[mmokhi]

databases/automysqlbackup/Makefile

30

Aha, yes.
Makes sense, I believe that's why we basically have the permission and ownership stuff in pkg-plist (as I suggested; more explanation available here).

Thanks for complete and reasonable explanation 🙏 👍

[me_cschwarz.com]

databases/automysqlbackup/Makefile

30

Any action required on my part? Otherwise please mark this as done.

databases/automysqlbackup/files/patch-automysqlbackup

10

When is %ETCDIR% replaced then? This is a patch file... Does the ports tree use some magic version of patch(1)?

[mmokhi]

databases/automysqlbackup/Makefile

30

I guess, It makes sense to update your patch deleting the chmod stuff from Makefile and handle all the rest in pkg-plist 😊
Then the rest is fine IMHO 👍

databases/automysqlbackup/files/patch-automysqlbackup

10

${REINPLACE_CMD} in post-patch target 😊
letting the patch file put the %ETCDIR% then you sed the patched file

[me_cschwarz.com]

• remove chmod from post-install (covered by pkg-plist)

[mmokhi]

The %ETCDIR% thingie I commented about might be as well good to take care of
(If people with paths different that /usr/local are using your port for example).
But I don't think it's a necessity at the moment 🙂
You can take care of it later 👍

[me_cschwarz.com]

Just replacing /usr/local/etc with %ETCDIR% doesn't work on my machine.
Is it because I override do-install?

[mmokhi]

In D19612#422465, @me_cschwarz.com wrote:

Just replacing /usr/local/etc with %ETCDIR% doesn't work on my machine.
Is it because I override do-install?

Sorry, maybe my comment was a bit misleading.
After replacing stuff with %ETCDIR% or anything in the patch-files, of course you need a post-patch: target with
a ${REINPLACE_CMD} in it to replace the %ETCDIR% value with ${ETCDIR}.

[me_cschwarz.com]

• use ETCDIR

[mmokhi]

Thanks.
To me it's perfect now 😊