https://bugzilla.redhat.com/show_bug.cgi?id=1309792
Ratnadeep Debnath rtnpro@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |needinfo?(jsmith.fedora@gma | |il.com)
--- Comment #3 from Ratnadeep Debnath rtnpro@gmail.com --- Hey Jared,
(In reply to Jared Smith from comment #2)
- Please add /etc/motdgen.d to the %files section, so that this package owns that directory
Fixed!
- Add dependencies for 'pam' and 'crontabs' and 'setup' if you're going to use the /etc/pam.d, etc/cron.daily, and /etc/profile.d directories
Fixed!
- /etc/cron.daily/motdgen-cache-updateinfo is marked as a config file, but
is executable
Most of the cron scripts are marked as executable. I asked @threebean about this, and he suggested to ignore this lint error.
I have pushed the updated SPEC and SRPM file at: Spec URL: https://rtnpro.fedorapeople.org/Packages/SPECS/fedora-motd.spec SRPM URL: https://rtnpro.fedorapeople.org/Packages/SRPMS/fedora-motd-0.1-1.fc22.src.rp...
Let me know what you think.