<html><head></head><body>I think all of the proposed changes sound good.<br>

<br>

Thanks<br>

-Ben<br><br><div class="gmail_quote">Dagobert Michelsen <dam@opencsw.org> wrote:<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<div style="white-space: pre-wrap; word-wrap:break-word; ">Fellow maintainers,

 as it was observed during the talk in Dublin some defaults in GAR are not really sane and it would be good to adjust them. The following variables are candidates for adjustment:

* MERGE_SCRIPTS_isa-extra = copy-relocated-only copy-config-only

This is copy-relocated-only at the moment. The present setting just copies

  $(bindir)/*-config but not the 64 bit equivalent

  $(bindir)/(sparcv9|amd64)/*-config

This change should be transparent to all packages only fixing issues about missing *-config for 64 bit.

* MERGE_DIRS_isa-extra = $(libdir)

This is now

  $(bindir) $(libdir) $(sbindir) $(libexecdir)

However, it was felt that most of the time only the libraries need to be provided for 64 bit and the binaries hence should be excluded for automatic inclusion.

Adding the old default to build recipes may or may not be good because there are several other ways to exclude files and it would be confusing to add them explcitily and later on remove them again.

* CONFIGURE_ARGS = $(DIRPATHS)

This defaults to empty ATM. The number of packages using configure with no arguments should be really small as this would be a non-standard build.

* TEST_TARGET = check

The previous target was "test". This can be detected and reset for Makefiles requiring the old target.

* NOISAEXEC = 1 -> ISAEXEC ?= 0

The current default is to use ISAEXEC when more then one ISA is built.

Again as most of the time libraries are used for multiple ISAs the defaulting of ISAEXEC is usually considered wrong and should be enabled together with $(bindir) MERGE_DIRS only when a benefit has been proven.

The current name is "NOISAEXEC", changing the default to "1" would require setting "NOISAEXEC = 0" which looks not straight forward.

I tend to invert it to default to "ISAEXEC ?= 0" and allow setting of "ISAEXEC = 1" in the Makefile. This can be detected and reset for Makefile using it.

* sysconfdir = /etc/opt/csw

* localstatedir = /var/opt/csw

These have been changed some time ago and should be the default now. Only very specific packages should stick to /opt/csw/etc as default and /opt/csw/var should not be used at all.

As the new default should apply to a lot of packages I would rather not update this automatically.

I plan to do the update in one large swoop.

Thoughts?

Best regards

  -- Dago<hr />maintainers mailing list

maintainers@lists.opencsw.org

<a href="https://lists.opencsw.org/mailman/listinfo/maintainers">https://lists.opencsw.org/mailman/listinfo/maintainers</a>

.:: This mailing list's archive is public. ::.

</div></blockquote></div></body></html>