NEW: lang/omega

[ggreif@…]

I am contributing my second port.

[ggreif@…]

Portfile and files/ (lang/omega/._Portfile is resource fork?)

[jmpalacios (Juan Manuel Palacios)]

I was ready to commit this port until I saw something a bit disturbing in the patchfile:

+PREFIX           = /usr/local

Which I cannot understand in light of destroot declaration in the Portfile:

destroot.args           PREFIX=${destroot}${prefix}

First off, hardcoding the prefix anywhere is bad practice. And even that put aside, choosing /usr/local as a prefix is even worse. What are each of these two lines for?

In case you want to provide a prefix declaration in the Makefile without hardcoding anything, a good approach is:

+PREFIX           = __PREFIX__

And then in the Portfile:

post-patch {
    reinplace "s/__PREFIX__/${prefix}/" path-to-file-you-want-to-edit
}

But in this case I just don't understand how prefix in the Makefile and destroot.args interplay. Please explain!

Regards,...

-jmpp

[ggreif@…]

The reason why I do it this way is simply because I do not know what the good practice is.

How do I tell to the install target in the makefile where to put the build result?

OTOH if I use $(PREFIX) in the makefile I consider it important to have a declaration of the make variable at least.

What is the recommended practice?

[heisenbug]

updated version

[ggreif@…]

new patched Makefile now makes use of DESTDIR var Portfile provides destroot.destdir, otherwise we are pretty standard now :-)

Please ignore lang/omega/._Portfile in the tarball!

[jmpalacios (Juan Manuel Palacios)]

Committed in r26579 with some minor fixes, see the commit log for details. Thanks for the contribution, keep it flowing!

-jmpp

[ggreif@…]

Any specific reason why you checked it in under lang/Omega instead of lang/omega?

(Almost) All the other ports use a small letter.

[jmpalacios (Juan Manuel Palacios)]

From looking at the project homepage I realized it's named Omega, not omega, so we should be consistent with that: I made both portname and portfir uppercase. I checked it in with the correct casing to begin with because modifying it later on when we realize "ohh, this should have been uppercase!" can be a pain. If there are ports that aren't consisten with their casing... then it'll be a pain renaming them ;-)

-jmpp

[jmpalacios (Juan Manuel Palacios)]

Milestone New Ports deleted

[(none)]

Milestone Port Submissions deleted