[kde-freebsd] port options are not saved in some ports

2013-07-07 Thread Andriy Gapon

It seems that port options do not "stick" in the following ports:
ukrainian/kde4-l10n
russian/kde4-l10n

Could you please take a look?

-- 
Andriy Gapon
___
kde-freebsd mailing list
kde-freebsd@kde.org
https://mail.kde.org/mailman/listinfo/kde-freebsd
See also http://freebsd.kde.org/ for latest information


Re: [kde-freebsd] port options are not saved in some ports

2013-07-07 Thread Baptiste Daroussin
On Mon, Jul 08, 2013 at 09:05:58AM +0300, Andriy Gapon wrote:
> 
> It seems that port options do not "stick" in the following ports:
> ukrainian/kde4-l10n
> russian/kde4-l10n
> 
> Could you please take a look?
> 
> -- 
> Andriy Gapon

That is because misc/kde4-l10n/files/bsd.l10n.mk is totally wrong

it does


...
.include 
...
.include 
...
.include 



First the right order in that case should be:

...
.include 
...
.include 
...
.include 


Second in that case (kde)
The pre.mk/post.mk can be removed just keeping the options.mk

and given that we now have the helpers options.mk can also be removed.

The check for variables in OPTIONS_DEFINE is also useless (already done by the
framework)

defining OPTIONS_DEFINE and OPTIONS_DEFAULT to some empty values is also
useless.

http://people.freebsd.org/~bapt/kde.diff

now just replace bsd.port.post.mk in russian/kde4-l10n/Makefile by bsd.port.mk
and everything will work properly again.

regards,
Bapt


pgpD20DMjFJgN.pgp
Description: PGP signature
___
kde-freebsd mailing list
kde-freebsd@kde.org
https://mail.kde.org/mailman/listinfo/kde-freebsd
See also http://freebsd.kde.org/ for latest information


Re: [kde-freebsd] port options are not saved in some ports

2013-07-09 Thread Raphael Kubo da Costa
Baptiste Daroussin  writes:

> http://people.freebsd.org/~bapt/kde.diff

This should be fixed in r322586, thank you very much.
___
kde-freebsd mailing list
kde-freebsd@kde.org
https://mail.kde.org/mailman/listinfo/kde-freebsd
See also http://freebsd.kde.org/ for latest information