[Bug 431665] Review Request: fox - A C++ library for GUI development

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: fox - A C++ library for GUI development


https://bugzilla.redhat.com/show_bug.cgi?id=431665





------- Additional Comments From pertusus@xxxxxxx  2008-03-08 03:46 EST -------
(In reply to comment #10)
> (In reply to comment #6)

> > The files in %_bindir also share a common namespace, so in 
> > there too they should be fox-PathFinder and fox-calculator.
> > (and corresponding man pages).
> > 
> 
> Upstream has advised me NOT to change it.

What are their arguments? The bindir is shared between all applications, 
the names of commands should be specific.

> fox-config can be dropped in fedora since it is mainly for other distro's.

If other upstream programs use fox-config it should be in fedora.
  
> > Regarding the include files there are very strange things, namely
> > there are autoconf conditionals in headers, like in fx3d.h
> > #ifdef HAVE_GLU_H
> > 
> > This is not right, these conditionals should be in .cpp files, 
> > not in the API.
> 
> The condition could probably be removed, if the classes not dependent
> on OpenGL are moved back to fx.h [FXVec** and so on].  These could
> be useful in a 2D environment w/o GL.

Whatever solution is used, the conditionals should go. An API should
never be conditional. But indeed part of the api could be factored
out in files not installed if that part of the api isn't compiled
in the library.

> > The file xincs.h is especially full of these, and also full of 
> > include files for other APIs. However it doesn't seems to be included
> > in any other file, so it may certainly be dropped completely from
> > the fox API.
> > 
> 
> This file is needed to write custom API's advised by upstream.

How can they advise that? Do they advise relying on autoconf 
conditionals? It is plain wrong.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]