Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 100025 - apache-2.0.54 fails to build with gcc-2.95.3
Summary: apache-2.0.54 fails to build with gcc-2.95.3
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Server (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Michael Stewart (vericgar) (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-07-23 06:40 UTC by Martin Schlemmer (RETIRED)
Modified: 2005-09-17 16:40 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
apache-2.0.54-gcc2.patch (apache-2.0.54-gcc2.patch,696 bytes, patch)
2005-07-23 06:40 UTC, Martin Schlemmer (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Schlemmer (RETIRED) gentoo-dev 2005-07-23 06:40:04 UTC
Some declarations in the middle of blocks of code cause apache to fail to build
on gcc2.
Comment 1 Martin Schlemmer (RETIRED) gentoo-dev 2005-07-23 06:40:23 UTC
Created attachment 64127 [details, diff]
apache-2.0.54-gcc2.patch

Patch to fix said issue.
Comment 2 Michael Stewart (vericgar) (RETIRED) gentoo-dev 2005-07-23 16:28:07 UTC
It looks like I introduced this with my patch to add AddDirectoryIndex and
RemoveDirectoryIndex.

I'm trying to figure out why changing where index_names is defined matters. 
Is this just a quirk with gcc 2.95.3, or is my code out of standard in some way,
or bad form? (This patch was one of my first tries at something more then a
simple "Hello World" in c/c++)
Comment 3 Martin Schlemmer (RETIRED) gentoo-dev 2005-07-24 01:23:14 UTC
OK, I did not check if it might be from a patch, as the damn p3 is slow ;p 
Anyhow, gcc2 do not support any declarations that is not at the start of {} or
global .. I think the c99 spec (or whatever) only started allowing this, and
thus the legality of it in gcc3+.
Comment 4 Michael Stewart (vericgar) (RETIRED) gentoo-dev 2005-09-17 16:40:55 UTC
Sorry, I forgot about this bug.

It's now been fixed in the SVN repository that our patch tarball is rolled from.
I plan on rolling a new tarball before new-style becomes stable (sometime tomorrow).

Resolving as fixed.