Please provide a license header in your spec file (an example can be found here: http://en.opensuse.org/openSUSE:Specfile_guidelines#Specfile_Licensing )

Rest looks good already, but I suggest to run "spec-cleaner" to get the package aligned with other packages already in openSUSE:Factory.