s1s0 / toped

Cross platform, open source IC layout editor
http://www.toped.org.uk/
GNU General Public License v2.0
15 stars 8 forks source link

toped fails to compile on Fedora Rawhide #149

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
toped package is failing to build in Fedora Rawhide. 

quadtree.cpp: In instantiation of 'bool 
laydata::ClipIterator<DataT>::secureNonEmptyDown() [with DataT = 
laydata::TdtData]':
quadtree.cpp:437:25:   required from here
quadtree.cpp:368:73: error: 'nextSubQuad' was not declared in this scope, and 
no declarations were found by argument-dependent lookup at the point of 
instantiation [-fpermissive]
quadtree.cpp:368:73: note: declarations in dependent base 
'laydata::Iterator<laydata::TdtData>' are not found by unqualified lookup
quadtree.cpp:368:73: note: use 'this->nextSubQuad' instead
quadtree.cpp: In instantiation of 'bool 
laydata::ClipIterator<DataT>::secureNonEmptyDown() [with DataT = 
auxdata::TdtAuxData]':
quadtree.cpp:438:25:   required from here
quadtree.cpp:368:73: error: 'nextSubQuad' was not declared in this scope, and 
no declarations were found by argument-dependent lookup at the point of 
instantiation [-fpermissive]
quadtree.cpp:368:73: note: declarations in dependent base 
'laydata::Iterator<auxdata::TdtAuxData>' are not found by unqualified lookup
quadtree.cpp:368:73: note: use 'this->nextSubQuad' instead
quadtree.cpp: In instantiation of 'bool 
laydata::DrawIterator<DataT>::secureNonEmptyDown() [with DataT = 
laydata::TdtData]':
quadtree.cpp:439:25:   required from here
quadtree.cpp:427:73: error: 'nextSubQuad' was not declared in this scope, and 
no declarations were found by argument-dependent lookup at the point of 
instantiation [-fpermissive]
quadtree.cpp:427:73: note: declarations in dependent base 
'laydata::Iterator<laydata::TdtData>' are not found by unqualified lookup
quadtree.cpp:427:73: note: use 'this->nextSubQuad' instead
quadtree.cpp: In instantiation of 'bool 
laydata::DrawIterator<DataT>::secureNonEmptyDown() [with DataT = 
auxdata::TdtAuxData]':
quadtree.cpp:440:25:   required from here
quadtree.cpp:427:73: error: 'nextSubQuad' was not declared in this scope, and 
no declarations were found by argument-dependent lookup at the point of 
instantiation [-fpermissive]
quadtree.cpp:427:73: note: declarations in dependent base 
'laydata::Iterator<auxdata::TdtAuxData>' are not found by unqualified lookup
quadtree.cpp:427:73: note: use 'this->nextSubQuad' instead
make[2]: *** [quadtree.lo] Error 1

see http://koji.fedoraproject.org/koji/taskinfo?taskID=4348316 for logs.  
Probably triggered by using gcc 4.7.1.

Original issue reported on code.google.com by opoplaw...@msn.com on 5 Aug 2012 at 3:05

GoogleCodeExporter commented 9 years ago
The attached patch allows quadtree.cpp to compile, but then got:

make[2]: Entering directory `/builddir/build/BUILD/toped-0.9.80/tpd_ifaces'
make[2]: *** No rule to make target `cif_yacc.h', needed by `all'.  Stop.
make[2]: Leaving directory `/builddir/build/BUILD/toped-0.9.80/tpd_ifaces'
make[1]: Leaving directory `/builddir/build/BUILD/toped-0.9.80'
make[1]: *** [all-recursive] Error 1

Original comment by opoplaw...@msn.com on 5 Aug 2012 at 3:39

Attachments:

GoogleCodeExporter commented 9 years ago
I'll have a look. Might need a bit of time to get a platform with latest gcc.

Original comment by krustev....@gmail.com on 5 Aug 2012 at 9:17

GoogleCodeExporter commented 9 years ago
OK, I did recreate the first problem on Fedora 17 (gcc 4.7.0 20120507)
I can confirm that the patch it appropriate and will be used to updated the 
trunk. I saw some other warnings which I would like to investigate as well.
That was the good news. 
The bad one is that I didn't see any further troubles like the one you quotes 
in the second post.
cif_yacc.h is a generated file (flex/bison) and the autotools/make should've 
taken care of it. I can't see the original error in the build system, the link 
you've sent contains only the first problem.
The suggestion is:
- I'll do some additional tests and will make a bugfix tarball in a couple of 
days time.
- Would you please double check what's the story with cif_yacc.h on the build 
site.

Original comment by krustev....@gmail.com on 6 Aug 2012 at 10:55

GoogleCodeExporter commented 9 years ago
New Linux packets are available for download

http://toped.googlecode.com/files/toped-0.9.8.1-r2211.tar.gz
http://toped.googlecode.com/files/toped-0.9.8.1-r2211.tar.bz2

Pleaswe let me know if they compile on rawhide

Original comment by krustev....@gmail.com on 11 Aug 2012 at 1:50

GoogleCodeExporter commented 9 years ago
Looks good.  Thanks!

Original comment by opoplaw...@msn.com on 11 Aug 2012 at 6:04

GoogleCodeExporter commented 9 years ago

Original comment by krustev....@gmail.com on 2 Oct 2012 at 10:26