launchpad-reviewers team mailing list archive

["Jeroen T. Vermeulen" <jtv@xxxxxxxxxxxxx>]

Jeroen T. Vermeulen has proposed merging lp:~jtv/maas/missing-template into lp:maas.

Commit message:
Fix "graft" line in MANIFEST.in so that it includes user_data.template like we meant it to.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~jtv/maas/missing-template/+merge/140586

As far as we can tell, "graft" operates on directories.  So the line "graft src/metadataserver/commissioning/*" would affect a directory such as src/metadataserver/commissioning/snippets/ but not src/metadataserver/commissioning/user_data.template, which is what it as meant for.  The python files (i.e. all the other files in src/metadataserver/commissioning/!) were being included anyway.


Jeroen
-- 
https://code.launchpad.net/~jtv/maas/missing-template/+merge/140586
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~jtv/maas/missing-template into lp:maas.

=== modified file 'MANIFEST.in'
--- MANIFEST.in	2012-12-18 17:06:43 +0000
+++ MANIFEST.in	2012-12-19 05:10:23 +0000
@@ -3,7 +3,7 @@
 graft src/*/fixtures
 graft src/*/specs
 graft src/provisioningserver/*
-graft src/metadataserver/commissioning/*
+graft src/metadataserver/commissioning
 graft vdenv
 prune src/*/testing
 prune src/*/tests