Page MenuHomeFreeBSD

New port: misc/paperwork: Need portmgr approval for flavors
Needs ReviewPublic

Authored by yuri on Jul 28 2018, 7:13 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jun 18, 4:41 PM
Unknown Object (File)
Fri, May 31, 3:17 AM
Unknown Object (File)
May 24 2024, 1:41 PM
Unknown Object (File)
May 10 2024, 10:02 PM
Unknown Object (File)
May 1 2024, 7:44 AM
Unknown Object (File)
Feb 18 2024, 3:02 PM
Unknown Object (File)
Feb 10 2024, 9:35 PM
Unknown Object (File)
Feb 8 2024, 4:18 PM
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None

Diff Detail

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

Event Timeline

misc/paperwork/Makefile
18–19

Wrong place in the Makefile. See Chapter 15. Order of Variables in Port Makefiles.

23–24

This is most likely wrong, I do not think gitlab account names can have / in them.

Also, as https://gitlab.gnome.org is not a gitlab instance, this should probably be:

GL_SITE=        https://gitlab.gnome.org/World
​GL_ACCOUNT=     OpenPaperwork
28–47

Please use flavor helpers, also, wrong place in the Makefile.

yuri marked 2 inline comments as done.Jul 30 2018, 5:17 PM

I think you need to add a description of the flavors in pkg-descr. Or even better, provide two pkg-descr files, one for each flavor, with a description of what part of the software they correspond to, exactly.

Add separate pkg-descr for each flavor.

yuri marked an inline comment as done.Aug 1 2018, 7:50 PM
yuri added inline comments.
misc/paperwork/Makefile
28–47

Not sure how to use helpers for these. Helpers aren't defined for PATCHDIR and WRKSRC_SUBDIR.

yuri marked 2 inline comments as done.Aug 1 2018, 7:50 PM
misc/paperwork/Makefile
28–47

Not sure what you mean. The lines this refers to are the depends lines.

Which you can see by clicking on the '<<' arrow meaning "this comment refers to a previous diff".

You should also use helpers for DESCR and PKGNAMESUFFIX.

yuri marked an inline comment as done.Aug 2 2018, 9:22 AM
yuri added inline comments.
misc/paperwork/Makefile
28–47

I added helpers for DESCR and PKGNAMESUFFIX.

yuri marked 2 inline comments as done.Aug 2 2018, 9:23 AM
yuri added inline comments.
misc/paperwork/Makefile
28–47

PATCHDIR doesn't work as a helper.

misc/paperwork/Makefile
28–47

Good, good, wondering if you are being obtuse on purpose or not.
Now, all you have to do is use helpers for the RUN_DEPENDS.

yuri marked an inline comment as done.Aug 2 2018, 5:55 PM
yuri added inline comments.
misc/paperwork/Makefile
28–47

Good, good, wondering if you are being obtuse on purpose or not.

I was wondering this way about you after every other message from you.

yuri marked 2 inline comments as done.Aug 2 2018, 5:55 PM