Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=728837 Yuguang Wang <yuwang@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Review Request: |Review Request: xml2dict - |python-xml2dict - Use |Use attributes of |attributes of dictionary to |dictionary to access xml |access xml elements. |elements. --- Comment #16 from Yuguang Wang <yuwang@xxxxxxxxxx> 2011-12-21 22:31:02 EST --- Spec URL: http://yuwang.fedorapeople.org/xml2dict.spec SRPM: http://yuwang.fedorapeople.org/xml2dict-0-0.1.2008.6.src.rpm (In reply to comment #11) I've also noticed this project before, but unfortunately it's nothing to do with http://code.google.com/p/xml2dict/ and I updated the license with GPLv2 according to the project homepage. > While looking into source code I realised that I have seen similar tarball > somewhere and found it at > https://nodeload.github.com/mcspring/XML2Dict/tarball/master > with its project page > https://github.com/mcspring/XML2Dict > > I see above project is updated and using license ASL 2.0 while your one is BSD BTW, Move package name from python-xml2dict back to xml2dict according to upstream, thanks for your suggestion. (In reply to comment #10) > - renaming the upstream tarball is not an option. Please include it as > xml2dict-2008.6-tar.gz > - Also you need to use the version specified by upstream and not use 1.0 as > version. You might use 0 as version and use 2009.6 in the release as documented > here: > http://fedoraproject.org/wiki/Packaging:NamingGuidelines#Pre-Release_packages Any other problems please let me know. Thanks. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review