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

Parag AN(पराग) <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |[email protected]
         AssignedTo|[email protected]    |[email protected]
               Flag|                            |fedora-review?

--- Comment #11 from Parag AN(पराग) <[email protected]> 2011-11-21 01:22:03 
EST ---
suggestions:
1) use source URL as
http://xml2dict.googlecode.com/files/xml2dict-%{alphatag}-tar.gz

and define

%global alphatag 2008.6

at top of your spec


2) use release tag as 0.%{1}.%{alphatag}


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

-- 
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
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to