Clumsy code it is :-) On Jun 26, 2013, at 6:27 PM, Jed Brown <[email protected]> wrote:
> Satish Balay <[email protected]> writes: > >> one more issue: >> >> looks like 2a571ffb5945aee713990b6770e04534a0423780 is not python-2.4 >> compliant. >> >> from configure_next_arch-linux-gcc-absoft_churn.log > > Damn, I checked that decorators were used elsewhere, but didn't actually > test with python-2.4. I now see that the other instances were not in > essential code. > > $ git grep '^\s*@' config > config/BuildSystem/config/setCompilers.py: @staticmethod > config/BuildSystem/config/setCompilers.py: @config.memoize > config/BuildSystem/patch.py: @property > config/BuildSystem/patch.py: @property > config/BuildSystem/patch.py: @property > config/builder.py: @staticmethod > config/builder.py: @staticmethod > config/builder.py: @staticmethod > config/builder.py: @property > config/builder.py: @property > > Should these other uses of decorators be replaced with the more clumsy > syntax? Or is it time to add --download-python?
