Page MenuHomeFreeBSD

lang/ruby34: Add new port
Needs ReviewPublic

Authored by yasu on Sat, May 18, 7:47 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, May 22, 3:28 AM
Unknown Object (File)
Wed, May 22, 3:24 AM
Unknown Object (File)
Wed, May 22, 2:48 AM
Unknown Object (File)
Tue, May 21, 5:13 PM
Subscribers

Details

Summary
Test Plan
  • Build succeeds with poudriere and 14.0-RELEASE amd64 jail.
  • Pass check of portlint, portclippy and portfmt.

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 57749
Build 54637: arc lint + arc unit

Event Timeline

yasu requested review of this revision.Sat, May 18, 7:47 AM
yasu created this revision.
diizzy added inline comments.
lang/ruby34/Makefile
18

${LOCALBASE} in LIB_DEPENDS in doesn't work?

40

I guess this falls under style preference but for consistency and logical reasons I'd place these variables before CONFIGURE_ARGS

43

Should be formatted so it's readable?

lang/ruby34/Makefile
18

Do you mean such line as below?

LIB_DEPENDD= ${LOCALBASE}/lib/libffi.so:devel/libffi

If so, unfortunately it doesn't.

40

Order of variables in Makefile is sorted with portclippy.

43

It is formatted with portfmt.