Re: [kde-freebsd] [patch] deskutils/kdeplasma-addons: Don't force kdeedu dependency

2009-09-15 Thread Mel
On Sunday 13 September 2009 03:09:56 Alberto Villa wrote: > On Sunday 13 September 2009 02:50:45 Mel wrote: > > I'll take a look, but I didn't on first glance see that the > > MacroOptionalDependency allows for a command line switch to disable it. > > if i remember correctly, you should be allowed

Re: [kde-freebsd] [patch] deskutils/kdeplasma-addons: Don't force kdeedu dependency

2009-09-12 Thread Alberto Villa
On Sunday 13 September 2009 02:50:45 Mel wrote: > I'll take a look, but I didn't on first glance see that the > MacroOptionalDependency allows for a command line switch to disable it. if i remember correctly, you should be allowed to use something like this: -DBUILD_$package:BOOL=Off have a look a

Re: [kde-freebsd] [patch] deskutils/kdeplasma-addons: Don't force kdeedu dependency

2009-09-12 Thread Mel
On Sunday 13 September 2009 00:30:02 Alberto Villa wrote: > On Saturday 12 September 2009 20:01:25 Mel Flynn wrote: > > deskutils/kdeplasma-addons forces dependency on the rather large package > > kdeedu, yet the CMakeLists.txt supports it as optional. > > > > Patch below doesn't change the defaul

Re: [kde-freebsd] [patch] deskutils/kdeplasma-addons: Don't force kdeedu dependency

2009-09-12 Thread Alberto Villa
On Saturday 12 September 2009 20:01:25 Mel Flynn wrote: > deskutils/kdeplasma-addons forces dependency on the rather large package > kdeedu, yet the CMakeLists.txt supports it as optional. > > Patch below doesn't change the default behavior, but allows user to set > WITHOUT_KDEEDU so that Kdeedu

[kde-freebsd] [patch] deskutils/kdeplasma-addons: Don't force kdeedu dependency

2009-09-12 Thread Mel Flynn
>Submitter-Id: current-users >Originator:Mel Flynn >Organization: >Confidential: no >Synopsis: [patch] deskutils/kdeplasma-addons: Don't force kdeedu >dependency >Severity: non-critical >Priority: low >Category: ports >Class: change-request >Release: Fre