Page MenuHomeFreeBSD

Move Mk/ to Mk/Uses/

Authored by acm on Mar 8 2018, 5:42 AM.
Referenced Files
Unknown Object (File)
Sun, May 19, 7:17 PM
Unknown Object (File)
Sun, May 19, 1:34 PM
Unknown Object (File)
Mon, May 13, 7:24 PM
Unknown Object (File)
Thu, May 9, 2:39 AM
Unknown Object (File)
Fri, May 3, 10:51 PM
Unknown Object (File)
Fri, Apr 26, 9:51 AM
Unknown Object (File)
Apr 22 2024, 5:54 PM
Unknown Object (File)
Apr 19 2024, 7:53 PM



It introduces Uses/ and move all fpc-ports to USES=fpc

Test Plan

Tested on head

Diff Detail

rP FreeBSD ports repository
Lint Not Applicable
Tests Not Applicable

Event Timeline

Could you use devel/arcanist, or at least generate a diff with full context like it does, with svn diff -x -U9999 or git diff -U9999.

42–50 ↗(On Diff #40064)

You should add DEV_WARNINGS if the WANT_* variables are set, so that they get removed.
You do not seem to be doing any checks to the content of fpc_ARGS, you should add some.

103 ↗(On Diff #40064)

Same here, USE_FPC=yes should give a warning.

203 ↗(On Diff #40064)

Why did you remove the _POST component? Is it never needed?

  • Add check of fpc_ARGS
  • USE_FPC warning is into
  • I think that _POST variable is not necessary because it doesn't rewrite any target
acm marked 2 inline comments as done.Mar 11 2018, 9:43 PM
224 ↗(On Diff #40179)

That can probably be removed.

acm marked an inline comment as done.
  • Remove line from end of file
10–12 ↗(On Diff #40197)

This is not required, there is a MAINTAINER line, any unauthorized commits can (and will) be backed out.

14–16 ↗(On Diff #40197)

This is a bit strange. It would probably be better to move the first sentence at the top, after "Support for FPC-...", and move the description of USE_FPC after the arguments description.

46–57 ↗(On Diff #40197)

As you already removed all uses of USE_FPC_RUN, WANT_FPC_BASE and WANT_FPC_ALL, so it would probably be best to remove support for them completely, and change the warnings into errors and put them in Mk/

1380–1383 ↗(On Diff #40197)

Unless I missed something, you added USES=fpc to all ports using USE_FPC, so this can be removed. (And the bit of doc about USE_FPC at the top of the file can go too.)

acm marked 4 inline comments as done.Mar 12 2018, 6:08 PM
  • Remove not necessary messages
  • Move WARNING message to Mk/
  • Remove lines about USE_FPC from Mk/
  • Some another changes

I cannot find anything more to add or ask, so...

I do not remember seeing an exp-run request for this, but be sure to ask for one before committing the patch.

  • Revert editors/lazarus to 1.8.0
This revision was not accepted when it landed; it landed in state Needs Review.Apr 5 2018, 4:52 AM
This revision was automatically updated to reflect the committed changes.