search

Qucs / qucs

Qucs Project official mirror
http://qucs.sourceforge.net/
GNU General Public License v2.0
1.16k stars 213 forks source link

travis strange behavior with Linux cmake Build #3159 #972

Closed oxmon-2500 closed 4 years ago

oxmon-2500 commented 4 years ago

Travis build terminates with

.adms.implicit.xml:743: unterminated attribute value - character " expected

The log statements appears double before failure:

[ 82%] Generating andor4x2.analogfunction.cpp, andor4x2.analogfunction.h, andor4x2.cpp, andor4x2.core.cpp, andor4x2.core.h, andor4x2.defs.h
[ 82%] Generating andor4x3.analogfunction.cpp, andor4x3.analogfunction.h, andor4x3.cpp, andor4x3.core.cpp, andor4x3.core.h, andor4x3.defs.h
[info...] admsXml-2.3.6 (e24d007) Feb 17 2020 18:38:04
[info...] admsXml-2.3.6 (e24d007) Feb 17 2020 18:38:04
[fatal..] .adms.implicit.xml:743: unterminated attribute value - character " expected

The PR didn't change anything in this area

in3otd commented 4 years ago

(this is about PR #971) Rather strange, I've restarted Travis there, let's see...

oxmon-2500 commented 4 years ago

The following run of Travis was OK (without changing anything).

guitorri commented 4 years ago

Perhaps Travis is building in 2 cores by default? Sometimes ADMS can run into race conditions during compilation between some generated files. There was a bug like this in the past. Not sure if it was not fixed on the develop branch of ADMS...

felix-salfelder commented 4 years ago

On Thu, Feb 20, 2020 at 07:13:11AM -0800, Guilherme Brondani Torri wrote:

Perhaps Travis is building in 2 cores by default? Sometimes ADMS can run into race conditions during compilation between some generated files. There was a bug like this in the past. Not sure if it was not fixed on the develop branch of ADMS...

ADMS is in Debian for quite some time now [1]. perhaps it is easiest to just use it in this context, save CPU cycles etc. Not sure what it takes to make current packages available within travis, should they not be already.

(the Debian package has been built in non-parallel mode, so it possibly still has the bug.)

[1] https://packages.debian.org/buster/adms