[Swig-devel] [ swig-Bugs-2985655 ] Regression: Renaming a constructor broken for python

[SourceForge.net <no...@so...>]

Bugs item #2985655, was opened at 2010-04-12 01:36
Message generated for change (Comment added) made by torsten
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=101645&aid=2985655&group_id=1645

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: python
Group: None
Status: Closed
Resolution: Fixed
Priority: 5
Private: No
Submitted By: Torsten Landschoff (torsten)
Assigned to: Haoyu Bai (bhy)
Summary: Regression: Renaming a constructor broken for python

Initial Comment:
While trying to build wxPython, I ran into this bug. The attached test case was broken by the Python3K support code merged in Revision 10834.



----------------------------------------------------------------------

Comment By: Torsten Landschoff (torsten)
Date: 2010-04-25 16:07

Message:
Thanks for fixing it so fast. :-)

----------------------------------------------------------------------

Comment By: Haoyu Bai (bhy)
Date: 2010-04-25 14:08

Message:
Fixed in latest commit. Thanks for the patch!

----------------------------------------------------------------------

Comment By: Torsten Landschoff (torsten)
Date: 2010-04-12 01:44

Message:
In case anybody ever cares (it could be me when hitting the next bug), this
is the script I used with git bisect. I also had to build a local bison
installation (I used the version from Debian lenny/stable) as otherwise
bison would cause the build to fail. Otherwise, it looks like every version
was built fine (from 1.3.32 onwards).

#! /bin/sh

set -x

cleanup() {
  rm -f Tools/config/config.guess Tools/config/config.sub
}
trap cleanup 0

./autogen.sh || exit 125
./configure --without-mzscheme --without-chicken || exit 125
rm -f swig
make -j5 || make || exit 125
rm -f `find Examples/test-suite -name constructor_rename_wrap.*`
rm -f `find Examples/test-suite -name constructor_rename.py`
make -C Examples/test-suite/python constructor_rename.cpptest
( 
  cd Examples/test-suite/python 
  python -c "import constructor_rename" || exit 125
  python constructor_rename_runme.py
)
code="$?"

if [ "$code" -eq 0 ]; then
  exit 0
fi

make clean-source
exit "$code"


----------------------------------------------------------------------

Comment By: Torsten Landschoff (torsten)
Date: 2010-04-12 01:40

Message:
Great, so the add button does not add the attachment but the bug entry.
:-)
I will add another attachment with the difference in the generated code.

This regression was introduced in 1.3.37 as it seems so it was not noticed
for quite a wile.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=101645&aid=2985655&group_id=1645