Kilian / f.lux-indicator-applet

This repository is unmaintained. Visit:
https://github.com/xflux-gui/xflux-gui
Other
549 stars 105 forks source link

Building RPM fails: name takes single token only #75

Open minorai opened 8 years ago

minorai commented 8 years ago

RPM building fails on Fedora 23, because the name value contains spaces.

building RPMs
error: line 7: Tag takes single token only: Name: f.lux indicator applet
error: query of specfile build/bdist.linux-x86_64/rpm/SPECS/f.lux indicator applet.spec failed, can't parse
error: Failed to execute: "rpm -q --qf '%{name}-%{version}-%{release}.src.rpm %{arch}/%{name}-%{version}-%{release}.%{arch}.rpm\\n' --specfile 'build/bdist.linux-x86_64/rpm/SPECS/f.lux indicator applet.spec'"

According to the distutils documentation the spaces should probably be avoided.

the distribution name is used to generate filenames, so you should stick to letters, digits, underscores, and hyphens.

It should probably be changed to f_lux/flux, flux-applet or similar