Add gchar* to xmlChar* casting of string constants to eliminate some

Derek Atkins warlord at MIT.EDU
Mon Jul 25 10:10:58 EDT 2005


Neil,

Neil Williams <linux at codehelp.co.uk> writes:

> On Monday 25 July 2005 2:10 am, David Hampton wrote:
>> Log Message:
>> -----------
>> Add gchar* to xmlChar* casting of string constants to eliminate some
>> gcc4 warnings.
>
> I have fixed these issues in the QOF commits but there are so many other 
> changes in both trees at the moment that I haven't been able to commit those 
> to GnuCash, sorry.

I think you need to be more proactive about patching GnuCash as well
as QOF-CVS.

> My apologies if the warnings are annoying during the build but these
> files are likely to change radically at the next commit. Any patches
> are very likely to be lost when GnuCash finally comes back into sync
> with QOF.

This is BAD.  Please make sure not to "lose patches" that are made in
the gnucash tree.  Yes, this is an issue with having two trees (or
worse, three or four).  This is all Linas' fault for pulling QOF out
before it was "ready".  ;)

Seriously, if you're going to work this way you have to collect
changes and apply them in both directions.  If a change is made in the
gnucash copy then you'll need to make sure that change isn't lost.

[snip]
> Anyway, if we can live with the inconvenience and warnings for a bit longer, 
> I'll commit the overhauled files, plus other changes in QOF, to GnuCash just 
> as soon as time allows.

Unfortunately it's more than just an inconvenience.  -Werror promotes
warnings to errors and causes the build to fail.

> Life will be so much easier (for me) when QOF is external.
> :-)

QOF being external is both a blessing and a curse..  It makes it MUCH
more difficult to fix bugs or make changes because you now have two
projects that need to release simultaneously.  It really slows down
the release cycle tremendously.

> Sorry for any inconvenience.

-derek

-- 
       Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory
       Member, MIT Student Information Processing Board  (SIPB)
       URL: http://web.mit.edu/warlord/    PP-ASEL-IA     N1NWH
       warlord at MIT.EDU                        PGP key available


More information about the gnucash-devel mailing list