New port: databases/jlog

[alfred]

Summary

Proposed commit message:

databases/jlog: New Journaled log port

JLog is short for "journaled log" and this package is really an API and implementation that is libjlog.
libjlog is a pure C, very simple durable message queue with multiple subscribers and publishers (both thread and multi-process safe).

WWW: https://labs.omniti.com/labs/jlog

DiffRev: https://reviews.freebsd.org/D3798
Submitted by: Alfred Perlstein (alfred@, maintainer)
Reviewed_by: 
Approved by:

Test Plan

• portlint -AC (include output)
• poudriere testport (upload or link poudriere log)

[alfred]

• portlint: regent distinfo and cleanup github directives
• shebang fixes

[brnrd]

Had to regenerate the distinfo{F253545} and modified the Makefile{F253549} to adapt to the new OPTIONS-helpers. Looks much cleaner I think.

• Update review's Summary and Test plan
• Update distinfo
• Remove GH_COMMIT, replace with GH_TAGNAME
• Add dependency on autoconf-2.13
• FIx shebangs (USES= shebangs)
• Fix perl packlist (USES= perl5)
• Refactor all conditionals to OPTIONS framework

Hope this helps you.

[alfred]

• Fix all the things (from brnrd@)
• Fixup for portlint -AC

[brnrd]

databases/jlog/Makefile

7

I'd guess 1.2.2 with a counter for the PORTREVISION would do just fine

22

# cf3085fe contains c++ fixes and other additions over the 1.2.2 release

36

Will ${PERL} always resolve to perl ? Otherwise it'll break if there's any chance that it'll be something other than configure expexts. If perl is on this is added to CONFIGURE_ARGS as --with-perl (and --without-perl when off)

41

This can be removed

[alfred]

OK about to resubmit.

databases/jlog/Makefile

7

Yes, but this isn't 1.2.2, it's a point in time release that we need. It's actually 1.2.2+commits until 201512200

22

ok

36

moving back to "perl" but portlint will complain

[alfred]

• Cleanup comment, remove extra include, fix PERL

[alfred]

• remove .packlist from plist, not needed

[brnrd]

LGTM

[koobs]

databases/jlog/Makefile

7

What does 's' stand for?

Might just a datestamp suffix suffice?

12

Add LICENSE_FILE if one exists in WRKSRC

14

Does it *actually* depend on and "call" a specific autoconf version-suffixed binary?

If not, set depends to version-less autoconf, OR what about USE_AUTOTOOLS=autoconf?

May be worth consider patching if currently checks for, but doesnt *require* 2.13.

19

If I recall correctly, PORTNAME is the default for GH_PROJECT.

25

USES=perl5, but PERL is an option?

Perhaps you meant USES=perl5:build only?

36

Are you sure you dont want to use USES=autoreconf here?

See if you can kill this custom pre-configure target. My guess is its not really needed.

[mat]

databases/jlog/Makefile

7

$ git describe cf3085f --tags
1.2.2-24-gcf3085f

There a 1.2.2 tag in the github repository, so, you would be better of setting DISTVERSION=1.2.2-24-gcf3085f

14

This feels so wrong.

27

There is only one option, PERL, and there is no %%PERL%% in the plist, you can drop this line.

36

Yeah, use USES=autoreconf.

36

portlint complains, it is what it does, it does not mean you have to blinly follow what it tells you :-)

[brnrd]

Closed by rP399111

[brnrd]

alfred@ long committed this to ports

[alfred]

This is committed.