After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 134886 - Compile fails on bzlib.h parse error before "FILE"
Compile fails on bzlib.h parse error before "FILE"
Status: RESOLVED FIXED
Product: yelp
Classification: Applications
Component: General
2.5.x
Other Linux
: Urgent major
: ---
Assigned To: Shaun McCance
Yelp maintainers
: 134903 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2004-02-19 19:28 UTC by Stef van der Made
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: 2.6.next
GNOME version: ---



Description Stef van der Made 2004-02-19 19:37:53 UTC
During the compile of garnome 0.30.1 I'm having some trouble compiling yelp
2.5.5 

Generating and caching the translation database
WARNING: ../po/th.po is not in UTF-8 but TIS-620, converting...
Merging translations into GNOME_Yelp.server.
In file included from yelp-io-channel.c:30:
/usr/include/bzlib.h:177: error: parse error before "FILE"
/usr/include/bzlib.h:205: error: parse error before "FILE"
make[4]: *** [yelp-io-channel.o] Error 1
make[4]: *** Waiting for unfinished jobs....
make[3]: *** [all-recursive] Error 1
make[2]: *** [all-recursive-am] Error 2
make[1]: *** [build-work/main.d/yelp-2.5.5/Makefile] Error 2
make: *** [paranoid-install] Error 2
Comment 1 Stef van der Made 2004-02-19 21:18:45 UTC
*** Bug 134903 has been marked as a duplicate of this bug. ***
Comment 2 Luis Villa 2004-02-26 21:31:21 UTC
This works fine here, admittedly with HEAD. Shaun, any idea?
Comment 3 Damon Chaplin 2004-03-09 13:56:13 UTC
Stef: can you check if your /usr/include/bzlib.h has an
'#include <stdio.h>' line in it.

Try adding '#include <stdio.h>' in yelp-io-channel.c, just before
the '#include <bzlib.h>'.
Comment 4 Stef van der Made 2004-03-11 20:17:15 UTC
Adding #include <stdio.h> to the yelp-io-channel.c file solved the
issue. Is this a problem only on my PC or will this include line be
added to the next version of yelp ?
Comment 5 Damon Chaplin 2004-03-17 15:53:13 UTC
I suspect you have a pretty old version of bzip2.

Adding '#include <stdio.h>' doesn't hurt anyway, so it should be added
to yelp.
Comment 6 Stef van der Made 2004-03-19 10:41:57 UTC
I've tried looking for a newer version but the latest is from DEC 2001
released jan2002. If there is a newer version somewhere that I've not
looked for please tell me.

Thanks
Comment 7 Shaun McCance 2004-03-19 19:16:25 UTC
I think it's pretty broken that a header file doesn't include the
headers it actually needs.  I'm not willing to break the code freeze
over this.  I'll add the include to yelp-io-channel after the release.

Stef: what kind of system is this?
Comment 8 Stef van der Made 2004-03-19 21:55:51 UTC
I'm running a slackware 8.0, with
gcc-3.3.3,glibc-.2.3.2,kernel-2.6.5-rc1 as the major updated components
Comment 9 Shaun McCance 2004-04-12 20:43:03 UTC
Updating milestone for 2.6.1.
Comment 10 Shaun McCance 2004-04-19 23:55:39 UTC
I've added #include <stdio.h> to yelp-io-channel.c on HEAD and gnome-2-6.  Closing.
Comment 11 Stef van der Made 2004-05-01 11:14:54 UTC
I've tried to compile yelp 2.6.1 and the problem still exists in the 2.6.1
source. As reported earlier. 
Comment 12 John Fleck 2004-05-01 19:59:38 UTC
Is the problem here that Shaun included the stdio.h *after* bzlib.h rather than
before? Stef, could you try moving it up? (And thanks for staying on this, Stef.)
Comment 13 Stef van der Made 2004-05-01 20:29:05 UTC
It looks like it. My include lines look like this when yelp compiles OK

#include <stdio.h>
#include <glib.h>
#include <zlib.h>
#ifdef HAVE_LIBBZ2
#include <bzlib.h>
#endif
#include <string.h>

Thanks for your prompt response.
Comment 14 Shaun McCance 2004-06-02 04:04:56 UTC
All right, sorry for the delay on this.  I've moved the stdio include above the
others on HEAD and gnome-2-6, so this should really be fixed now.  Feel free to
reopen if there's still a problem.