A couple of cvs nitpicks

Carl Petersen xfce at mostlysoftware.com
Thu Mar 4 16:50:34 CET 2004


Jasper Huijsmans wrote:
> On Thu, Mar 04, 2004 at 10:16:52AM -0500, Carl Petersen wrote:
> 
>>Couple of minor nit-picks in cvs:
>>
>>In xfdesktop; settings/backdrop-icon.h is not deleted by distclean and 
>>it should be.
> 
> I don't think I agree, what's the point in regenerating it every time?

Your right, it's just that `cvs update` questions its existence since 
it's not in cvs. Not a problem.

>>In sample-plugin; sample.c contains a function called 
>>"create_options()", should be "sample_create_options()".
> 
> 
> Well, the sample plugin is not being maintained very thoroughly, so
> there are probably many more problems with it. If I remember, I'll fix
> this one. That reminds me, we should probably override the install target, 
> to prevent people from actually using it ...

Seems to work fine, a good example.

> Also, these issues are probably better discussed on xfce4-dev.

Sorry, I'll post there instead.
> 
> 	Jasper
> _______________________________________________
> Xfce mailing list
> Xfce at xfce.org
> http://lunar-linux.org/mailman/listinfo/xfce
> http://www.xfce.org
> 



More information about the Xfce mailing list