[Frugalware-devel] fblint and depends

[Gabriel C]

VOROSKOI Andras wrote:
> Hi guys,
> 
> Today i got a bugreport from hermier@, he noticed that ntp need rebuilding
> with new openssl. Then i've realized that ntp do not have a depends=()
> line in its FB.
> 
> So i wrote a tiny patch for fblint, but:
> 
> Now it do not allow empty depends. So you have to use depends=(' ') at
> least if you mean that the given package does not have any depends.
> 
> Our choiches are:
> 	1) voroskoi's patch is a bullshit, that guy must be mentally
> handicapped -> revert
> 	2) using depends=(' ') instead of depends=('')
> 	3) using # no depends() in FB (so a commit message)

Hehe :P should be :

Our choiches are:
        1) voroskoi's patch is a bullshit, that guy must be mentally handicapped -> revert

If you think is not what style of 'depends' check should be used :
 	1a) using depends=(' ') instead of depends=('')
 	1b) using # no depends() in FB (so a commit message)
	1c) other ideas ?
 
> I like 2, but VMiklos sais it's stupid to set an empty variable and that's
> also true. Please vote, which one do you prefer (or let us know your
> earthbreaking idea)!

I don't think depends=(' ') is stupid the result is the same anyway.

I also think your patch is ok , we should check for depends= in FB's.

That said I'm for whatever solution is adding actually an check but I'm not really sure about the 'style'
Anyway your 2) or 1a) :p is fine for me.

> 
> Good night!
> 

gn :)


crazy