Page MenuHomeFreeBSD

New port: databases/jlog
AbandonedPublic

Authored by alfred on Oct 4 2015, 6:18 PM.
Tags
None
Referenced Files
F132225804: D3798.id9302.diff
Tue, Oct 14, 11:31 PM
Unknown Object (File)
Tue, Oct 14, 2:46 PM
Unknown Object (File)
Tue, Oct 14, 2:45 PM
Unknown Object (File)
Tue, Oct 14, 2:45 PM
Unknown Object (File)
Tue, Oct 14, 2:45 PM
Unknown Object (File)
Tue, Oct 14, 2:45 PM
Unknown Object (File)
Tue, Oct 14, 2:39 AM
Unknown Object (File)
Tue, Oct 14, 1:09 AM
Subscribers

Details

Reviewers
koobs
mat
Group Reviewers
Ports Committers
Perl
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)

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 744
Build 744: arc lint + arc unit

Event Timeline

alfred retitled this revision from to New port: databases/jlog.
alfred updated this object.
alfred edited the test plan for this revision. (Show Details)
  • portlint: regent distinfo and cleanup github directives
  • shebang fixes
brnrd edited the test plan for this revision. (Show Details)

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.

  • Fix all the things (from brnrd@)
  • Fixup for portlint -AC
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

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

  • Cleanup comment, remove extra include, fix PERL
  • remove .packlist from plist, not needed
brnrd edited edge metadata.

LGTM

This revision is now accepted and ready to land.Oct 11 2015, 6:34 PM
koobs requested changes to this revision.Oct 13 2015, 11:54 AM
koobs edited edge metadata.
koobs added inline comments.
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.

This revision now requires changes to proceed.Oct 13 2015, 11:54 AM
mat requested changes to this revision.Oct 13 2015, 2:00 PM
mat added a reviewer: mat.
mat added a subscriber: mat.
mat added inline comments.
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 removed a reviewer: brnrd.

alfred@ long committed this to ports