Created attachment 222500 [details]
Use PORTNAME & COMMENT to easily copy & paste files between ports.

see NOTES above.

Created attachment 222501 [details]
Fixes compiler warnings in the source *.c files.

Comment 3 Fernando Apesteguía 2021-02-18 08:06:24 UTC

^Triage: Submitter is not maintainer, can not set maintainer-feedback to '+'

Created attachment 222554 [details]
Patch to make pkg-plist variables work better

Hello,

I approve the patches. 

Thanks Walter, for tidying-up the port.

But I think the pkg-plist handling is not complete according to https://docs.freebsd.org/en/books/porters-handbook/#plist

I have added a pkg-plist.diff to fix this. Apply on top of the other patches.

Please have a look if everything is ok (worked for me :-) )

Thanks

-Derik.

(In reply to Derik van Zuetphen from comment #5)
OK, it runs all through w/o errors.  I'm wondering what happens with all those *.orig and empty files.  But since the ports are built in poudriere, I guess they do not hurt and/or you'll take care of that.

Test: Now I suspend... and back again :)
Feb 19 00:09:16 t450s root[48107]: /etc/rc.resume: INFO: starting anacron after resume
Feb 19 00:09:16 t450s anacron[49721]: Anacron 2.3 started on 2021-02-19
Feb 19 00:09:16 t450s anacron[49721]: Will run job `ntpdate' in 1 min.
Feb 19 00:09:16 t450s anacron[49721]: Will run job `periodic-daily' in 5 min.
Feb 19 00:09:16 t450s anacron[49721]: Jobs will be executed sequentially

I can't see the fix in the updates of the /latest/ ports branch (I have both latest & quarterly).  What can I do to help get that done?  Was it that I changed the state to "fixed/CLOSED" is preventing an update?  Then it's a bug in this web interface: it should not allow me to set such flags when I'm not supposed to do that.  Maybe it's just that the maintainer is busy with other stuff, and we'll see an update in a week or two, that's fine with me.

Should I file in a bug report on that a bug reporter should not be allowed to set "maintainer approval"?  Now the description was updated to be more precise?  Seems I misunderstood this. "Set (?) to request MAINTAINER approval"  Since the maintainer's e-mail is known to the system, I found it logical the system asks for my e-mail address here (although that's known, too). My english is not overly bad (english = ancient gaelic? + ancient german with latin grammatics, loosely speaking), so this must have been much more misleading for others whose native tongue is further away from english.

Comment 8 Fernando Apesteguía 2021-03-04 12:32:51 UTC

(In reply to Walter von Entferndt from comment #7)

Yes, probably. A committer might have passed by and omit this PR since it was already closed. Let's change it to Open and see if someone picks it up.

I (maintainer) can add a patch with all changes incorporated, if that helps.

Just tell me.

-Derik.

Comment 10 Fernando Apesteguía 2021-03-04 15:20:29 UTC

(In reply to Derik van Zuetphen from comment #9)
That would probably help: adding a new patch with everything and obsoleting the old ones.

I would commit this myself, but I don't have any means to test this. Another committer might be more suitable.

(In reply to Fernando Apesteguía from comment #10)
I would have provided a clean up (combination) of the patches, but I don't have all the ports maintainer's tools installed, only the std ones like diff & patch.  I saw that e.g. Derik was using a special git-diff or such, IIRC.  Thus I thought I leave it as is & the maintainer takes care of the rest.  Learning all the time: next topic -> TODO list: install ports-maintainers tools to provide better patches in bug reports.

On testing: I can affirm that the combined patches build a working executable & service script that starts anacron @resume.  It runs fine on my machine 12.1-REL now 12.2-REL since Feb. 18th, 2021.  Have to reboot much less than before, fine.

Created attachment 223000 [details]
All patches combined

Hello,

I have made a combined path of all patches mentioned above.

Please Committer take over :-)

Thank you
-Derik.

Can a committer have a look at the patches and commit them ....

Thanks

ping

What can I do to help to get this fix commited?  I can only tell it works for me... and the port's maintainer is satisfied, too.  What else is missing?

Comment 15 Fernando Apesteguía 2021-03-30 14:26:17 UTC

(In reply to Walter von Entferndt from comment #14)
OK, I'll take it. What is it closed Walter?

(In reply to Fernando Apesteguía from comment #15)
I closed it because I felt the need to do SOMETHING...  dunno what that does in the workflow.  All what was written is true & not hoax, was tested on FreeBSD 12.2-RELEASE-p5 (now) & I'd be gratefull if this fix eventually gets committed :)

Comment 17 Fernando Apesteguía 2021-03-31 05:37:51 UTC

===> Checking for items in STAGEDIR missing from pkg-plist
Error: Orphaned: @dir /var/spool/%%PORTNAME%%

This part was missing from pkg-plist. No need to upload a new patch. I will commit this from git. The ports collection is now frozen due to the svn->git transition.

Thanks!

Comment 18 commit-hook 2021-04-07 11:23:56 UTC

A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=1a76a776c71cf0cf74e8e66586cc64cc6896b558

commit 1a76a776c71cf0cf74e8e66586cc64cc6896b558
Author:     Fernando Apesteguía <fernape@FreeBSD.org>
AuthorDate: 2021-04-07 11:16:00 +0000
Commit:     Fernando Apesteguía <fernape@FreeBSD.org>
CommitDate: 2021-04-07 11:20:03 +0000

    sysutils/anacron: Fix start after resume

    The anacron utility should be run by it's rc(8) script after resume, since the
    time to run jobs from the anacrontab(5) might have passed while the system was
    sleeping.

    PR:     253567
    Reported by:    walter.von.entferndt@posteo.net
    Approved by:    dz@426.ch (maintainer)

 sysutils/anacron/Makefile                 | 20 +++++++++++---------
 sysutils/anacron/files/anacron.in         | 31 +++++++++++++++++++++++--------
 sysutils/anacron/files/patch-log.c (new)  | 10 ++++++++++
 sysutils/anacron/files/patch-main.c (new) | 10 ++++++++++
 sysutils/anacron/files/patch-matchrx.c    |  6 +++++-
 sysutils/anacron/files/patch-runjob.c     | 13 +++++++++++--
 sysutils/anacron/files/pkg-message.in     | 15 ++++++++++-----
 sysutils/anacron/pkg-plist                | 12 +++++++-----
 8 files changed, 87 insertions(+), 30 deletions(-)

Comment 19 Fernando Apesteguía 2021-04-07 11:24:26 UTC

Committed,

Thank you both for your patience!