Skip to main content
Sign in
Snippets Groups Projects
Commit 753e43b9 authored by Kevin Lyda's avatar Kevin Lyda :speech_balloon:
Browse files

Merge pull request #3 from overplumbum/master

Accept filenames used in Debian packaging.

According to the Debian packaging guides, files named debian/foo.cron.d will be installed as /etc/cron.d/foo. Files named debian/cron.d are also allowed - the cron.d file is given the package name in /etc/cron.d. For more info: http://www.debian.org/doc/manuals/maint-guide/dother.en.html#crond

In order to check these files during the packaging phase, files matching (\S+\.)?cron.d must be allowed.

At a later point it is probably a good idea to have these filename extensions enabled by flags. As the comment preceding the code alludes to, some names that run-parts would not accept are already allowed for another file deployment system.

I'll do that later after I review some other packaging system docs.
parents e91e1215 f8c4b246
Branches
Tags
No related merge requests found
...@@ -84,7 +84,7 @@ USER_WHITELIST = set(('postgres', 'buildbot', ...@@ -84,7 +84,7 @@ USER_WHITELIST = set(('postgres', 'buildbot',
# The following extensions imply further postprocessing or that the slack # The following extensions imply further postprocessing or that the slack
# role was for a cron that allowed dots in cron scripts. # role was for a cron that allowed dots in cron scripts.
FILE_RE_WHITELIST = [re.compile(x) for x in FILE_RE_WHITELIST = [re.compile(x) for x in
('\.in$', '\.cron$', '\.disabled$')] (r'\.in$', r'\.cron$', r'\.disabled$', r'^(\S+\.)?cron\.d$')]
class FSM(object): class FSM(object):
... ...
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please to comment