Actions
Bug #6460
closed
Architecture #6410: Make ncf api compatible with python 3
Typo in ncf.py, cannot be used
Status:
Rejected
Priority:
1 (highest)
Assignee:
Category:
Generic methods
Target version:
Pull Request:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
Fix check:
Regression:
Description
A typo was inserted in tool/ncf.py preventing it usage
line 140:
if sys.version_info.major == 2:
unicodeLine = unicode(line,"UTF-8") #line.decode('unicode-escape')
else
unicodeLine = line # = str(line, "UTF-8")
The else is missing a colon
Updated by Vincent MEMBRÉ over 9 years ago
- Status changed from New to In progress
Updated by Vincent MEMBRÉ over 9 years ago
- Status changed from In progress to Pending technical review
- Assignee changed from Vincent MEMBRÉ to Benoît PECCATTE
- Pull Request set to https://github.com/Normation/ncf/pull/185
Updated by Vincent MEMBRÉ over 9 years ago
- Status changed from Pending technical review to Rejected
- Assignee changed from Benoît PECCATTE to Vincent MEMBRÉ
- Pull Request deleted (
https://github.com/Normation/ncf/pull/185)
In fact it will be fixed by #6440; rejecting this and making the technical review !
Updated by Vincent MEMBRÉ over 9 years ago
- Related to Bug #6440: We can't use é in the generic methods parameters added
Updated by Alexis Mousset over 2 years ago
- Target version changed from 0.x to ncf-0.x
- Priority set to 0
Actions