Details
Diff Detail
- Repository
- rP FreeBSD ports repository
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
games/fretsonfire/Makefile | ||
---|---|---|
5 ↗ | (On Diff #37666) | While I agree that the port probably should have been called that when it was created, it wasn't. Also you don't seem to gain much by doing it, you still need to sed the variable, just at different places. |
games/fretsonfire/Makefile | ||
---|---|---|
5 ↗ | (On Diff #37666) | This ${PORTNAME:S/f/F/g:S/o/O/} was repeated 5 times! And it is not a nice thing, it's hackish. It became more simple and readable, IMO. |
games/fretsonfire/Makefile | ||
---|---|---|
5 ↗ | (On Diff #37666) | Then simply set some variable to ${PORTNAME:S/f/F/g:S/o/O/} and use it. Or set a variable to FretsOnFire and be done with it. |
How important is it to keep the original lowercase PORTNAME?
This is an obscure port, and PORTNAME should have obviously been mixed-case FretsOnFire.
This port is also breaking in build for a very long time.
So, why not just change it now to what it should be, especially that it simplifies things, and also get rid of that ugly hack?
Well, it's not that important, to me it's just ugly to have the two changes
- port foo/barbaz -> package BarBaz-123
- barbaz-122 updated to BarBaz-123
so if you could fix the build without doing that, that would be great(er) ^^