Page MenuHomeFreeBSD

Convert Mk/bsd.octave.mk to Mk/Uses/octave.mk
ClosedPublic

Authored by bofh on Dec 28 2022, 12:03 AM.
Referenced Files
Unknown Object (File)
Wed, Mar 6, 12:05 AM
Unknown Object (File)
Wed, Mar 6, 12:05 AM
Unknown Object (File)
Wed, Mar 6, 12:05 AM
Unknown Object (File)
Wed, Mar 6, 12:05 AM
Unknown Object (File)
Wed, Mar 6, 12:05 AM
Unknown Object (File)
Tue, Mar 5, 11:31 PM
Unknown Object (File)
Sat, Mar 2, 10:43 AM
Unknown Object (File)
Feb 16 2024, 8:36 PM
Subscribers

Details

Summary

There is a bigger patch including all the changes for the consumers but I have just submitted the Mk framework patches. I am still unsure whether if I should make one big commit or should I make individual commits to convert to the new macro USES=octave.

Test Plan

This is the last run of the consumers using the new USES macro:
~~ https://pdr.bofh.network/build.html?mastername=131-tex&build=2022-12-27_14h34m43s ~~
https://pdr.bofh.network/build.html?mastername=131-tex&build=2022-12-29_15h48m22s

Unless I have missed any consumers all builds fine.

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

bofh requested review of this revision.Dec 28 2022, 12:03 AM
bofh created this revision.

Is it intentional that we request c++14 support but set gnu++11 in CXXFLAGS?
L1 and L2 should be removed and replaced with a short description instead?

Is it intentional that we request c++14 support but set gnu++11 in CXXFLAGS?

Nah prehistoric garbage.

L1 and L2 should be removed and replaced with a short description instead?

If you delete the L1 and L2 the features and description comes on top. :v

  • Remove CXXFLAGS=-std-gnu++11
  • Remove editorconfigs

I think as this replaces bsd.octave.mk, I would push it and the conversion of the tree in one big commit.

Mk/Uses/octave.mk
17

^ this looks the wrong way around.

In my opinion the mk should provide the version and the Makefile get it from here.

37

^ do all octave-using ports require this, if not maybe add an argument :nosubdir?

51

maybe define octave-preinstall, octave-do-install and ocatave-post-install and add it?
ala

pre-install: octave-pre-install

?

Mk/Uses/octave.mk
37

Yes this is required by all consumers.

bofh edited the test plan for this revision. (Show Details)

This includes the entire BIG patch to convert bsd.octave.mk to Mk/Uses/octave.mk including the updates requested by @tcberner

This looks good to me.

math/octave-forge-divand/Makefile
21 โ†—(On Diff #114643)

^ah, so there is at least one exception to the WRKSRC :D

This revision is now accepted and ready to land.Dec 30 2022, 9:00 AM
math/octave/Makefile
66 โ†—(On Diff #114643)

^ unrelated, could likely be USES=localbase

This revision was automatically updated to reflect the committed changes.