Page MenuHomeFreeBSD
Differential D16523

Add devel/hs-fortran-src port - a library for lexing and parsing Fortran source code.
ClosedPublic
Actions

Authored by arrowd on Jul 31 2018, 8:59 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jan 18, 8:15 AM
Unknown Object (File)
Tue, Jan 7, 2:47 PM
Unknown Object (File)
Tue, Jan 7, 2:43 PM
Unknown Object (File)
Thu, Jan 2, 3:42 PM
Unknown Object (File)
Dec 4 2024, 3:46 PM
Unknown Object (File)
Nov 1 2024, 7:16 AM
Unknown Object (File)
Oct 30 2024, 10:10 AM
Unknown Object (File)
Oct 23 2024, 11:49 AM
View All Files
Subscribers
mat

Details

Reviewers
tcberner
AMDmi3
Group Reviewers
Contributor Reviewers (ports)
Commits
rP476202: Add devel/hs-fortran-src port - a library for lexing and parsing Fortran source…

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

arrowd created this revision.Jul 31 2018, 8:59 AM
Harbormaster completed remote builds in B18453: Diff 46062.Jul 31 2018, 8:59 AM
mat added a subscriber: mat.Jul 31 2018, 9:49 AM
mat added inline comments.
devel/hs-GenericPretty/Makefile
1 ↗(On Diff #46062)

Missing the header.

devel/hs-GenericPretty/pkg-descr
4 ↗(On Diff #46062)

Remove.

devel/hs-fortran-src/Makefile
1 ↗(On Diff #46062)

Missing header.

devel/hs-fortran-src/pkg-descr
4 ↗(On Diff #46062)

Remove.

arrowd added inline comments.Jul 31 2018, 8:57 PM
devel/hs-GenericPretty/Makefile
1 ↗(On Diff #46062)

What header? $ FreeBSD $?

devel/hs-GenericPretty/pkg-descr
4 ↗(On Diff #46062)

Newline or WWW line?

tcberner added inline comments.Aug 1 2018, 5:29 AM
devel/hs-GenericPretty/Makefile
1 ↗(On Diff #46062)

Yup, that one is always required, see sample Makefile

devel/hs-GenericPretty/pkg-descr
4 ↗(On Diff #46062)

new line :)

tcberner added a comment.Aug 1 2018, 5:30 AM

When adding new [or removing] ports, you always need to carry the changes in the category Makefile's SUBDIR list.

tcberner added inline comments.Aug 1 2018, 5:32 AM
devel/hs-GenericPretty/Makefile
3 ↗(On Diff #46062)

^ new ports should not have a portrevision

6 ↗(On Diff #46062)

^ is that the correct email? :D

mat added inline comments.Aug 1 2018, 7:43 AM
devel/hs-GenericPretty/pkg-descr
4 ↗(On Diff #46062)

My comment is related to line 4, so, yes, blank line.

arrowd updated this revision to Diff 46122.Aug 1 2018, 8:54 AM
arrowd marked 11 inline comments as done.

Address comments.

Harbormaster completed remote builds in B18478: Diff 46122.Aug 1 2018, 8:54 AM
arrowd added inline comments.Aug 1 2018, 8:54 AM
devel/hs-GenericPretty/Makefile
6 ↗(On Diff #46062)

Yes, the port was submitted by other guy.

mat added inline comments.Aug 1 2018, 9:05 AM
devel/hs-GenericPretty/pkg-descr
1 ↗(On Diff #46122)

pkg-descr must be a few lines long, at least a paragraph.

devel/hs-fortran-src/pkg-descr
1 ↗(On Diff #46122)

I do not think anlyses is a word.
Also, pkg-descr must be a few lines long, at least a paragraph.

tcberner added inline comments.Aug 1 2018, 12:00 PM
devel/hs-fortran-src/pkg-descr
1 ↗(On Diff #46122)

I had the same reaction -- it's the description of the thingy on it's website.

Further down they spell it more conventionally as analyses.

tcberner requested changes to this revision.Aug 1 2018, 12:01 PM

You're still missing the entries in devel/Makefile.

This revision now requires changes to proceed.Aug 1 2018, 12:01 PM
arrowd updated this revision to Diff 46138.Aug 1 2018, 6:04 PM
arrowd marked 2 inline comments as done.

Address comments.

Harbormaster completed remote builds in B18488: Diff 46138.Aug 1 2018, 6:04 PM
tcberner accepted this revision.Aug 1 2018, 7:02 PM

Looks good to me.

This revision is now accepted and ready to land.Aug 1 2018, 7:02 PM
Closed by commit rP476202: Add devel/hs-fortran-src port - a library for lexing and parsing Fortran source… (authored by arrowd). · Explain WhyAug 1 2018, 9:53 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
devel/
2 lines
hs-GenericPretty/
13 lines
3 lines
5 lines
hs-fortran-src/
19 lines
3 lines
6 lines
lang/
ghc/
2 lines

Diff 46163

View Options

head/devel/Makefile

Loading...
View Options

head/devel/hs-GenericPretty/Makefile

Loading...
View Options

head/devel/hs-GenericPretty/distinfo

Loading...
View Options

head/devel/hs-GenericPretty/pkg-descr

Loading...
View Options

head/devel/hs-fortran-src/Makefile

Loading...
View Options

head/devel/hs-fortran-src/distinfo

Loading...
View Options

head/devel/hs-fortran-src/pkg-descr

Loading...
View Options

head/lang/ghc/bsd.hackage.mk

Loading...