Rich Burridge wrote:

>   http://jurassic.us.oracle.com/~richb/7085723-v1/

Most of this is fine.  But for the components which do more than simply set
human-version to $(COMPONENT_VERSION), I'd rather see something similar to
what you did for aalib -- define a HUMAN_VERSION macro, and build it up in
the Makefile, rather than in the manifest.  So, for imagemagick:

    Makefile:
        HUMAN_VERSION=$(COMPONENT_VERSION)-$(COMPONENT_SUBVERSION)
        PKG_MACROS += HUMAN_VERSION=$(HUMAN_VERSION)
    imagemagick.p5m:
        set name=pkg.human-version value=$(HUMAN_VERSION)

In fact, you could probably just put HUMAN_VERSION into PKG_MACROS up in
ips.mk.  I think the others are ntp, proftpd, and slrn.

Thanks,
Danek
_______________________________________________
userland-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/userland-discuss

Reply via email to