On 26/01/11 12:25, Anders Logg wrote: > On Wed, Jan 26, 2011 at 12:34:11PM +0100, Marie E. Rognes wrote: >> On 01/25/2011 10:24 PM, Johan Hake wrote: >> >> Looks silly, but it seams like the error was introduced after: >> >> http://bazaar.launchpad.net/~dolfin-core/dolfin/main/revision/5570 >> >> >> This is a bit interesting. The buildbot indeed reports the error after >> revision >> 5570: >> >> >> • Project: dolfin >> • Revision: 5570 >> • Author: Garth N. Wells >> • Date: Mon 2011-01-24 13:13:40 +0000 >> • Files: >> □ dolfin/fem/LinearVariationalSolver.cpp >> >> Comments: >> >> Fix bug in LinearVariationalSolver. >> >> However, clicking on 5570, takes you to the launchpad site with >> >> >> Anders Logg >> >> 2011-01-24 13:43:49 >> >> Revision ID: [email protected] >> >> Only extract mesh from coefficients when no other mesh is present >> since it may otherwise conflict with existing meshes if coefficient >> is defined on a different mesh. Should fix buildbot error. >> >> >> This is probably because of a merge, but can be rather confusing ... > > Looks like it was partly my fault (for sending the wrong arguments to > assemble in LinearVariationalSolver) and partly Garth's fault (for > replacing my wrong arguments with another set of wrong arguments)... :-) >
I'd say the fault belongs with Anders - there were two errors, and I only fixed one of them the first time around ;). Garth > -- > Anders > > _______________________________________________ > Mailing list: https://launchpad.net/~dolfin > Post to : [email protected] > Unsubscribe : https://launchpad.net/~dolfin > More help : https://help.launchpad.net/ListHelp _______________________________________________ Mailing list: https://launchpad.net/~dolfin Post to : [email protected] Unsubscribe : https://launchpad.net/~dolfin More help : https://help.launchpad.net/ListHelp
