Summary: | [GTK] Do not include files that are not in git in the tarball | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Carlos Garcia Campos <cgarcia> | ||||||
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | gustavo, svillar | ||||||
Priority: | P2 | Keywords: | Gtk | ||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Carlos Garcia Campos
2014-07-10 09:56:35 PDT
Created attachment 234708 [details]
Patch
Comment on attachment 234708 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=234708&action=review > Tools/gtk/make-dist.py:94 > + self.files_in_cms = self.list_cms_files() I'd prefer files_ins_vcs or files_in_version_control > Tools/gtk/make-dist.py:104 > + cmd = ['git', 'ls-tree', '-r', '--name-only', 'HEAD', self.source_root] > + p = subprocess.Popen(cmd, stdout=subprocess.PIPE) This could be one line. This doesn't really support SVN. Is there a script in the WebKit scripts that can do this for us? > Tools/gtk/make-dist.py:129 > - if not passes_all_rules(file): > + if not passes_all_rules(file) or self.should_skip_file(file): Is this going to do the right thing for files from the build directory? (In reply to comment #2) > (From update of attachment 234708 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=234708&action=review > > > Tools/gtk/make-dist.py:94 > > + self.files_in_cms = self.list_cms_files() > > I'd prefer files_ins_vcs or files_in_version_control I actually meant scm :-P which is what webkitpy uses, but I don't really mind either vcs or version_control > > Tools/gtk/make-dist.py:104 > > + cmd = ['git', 'ls-tree', '-r', '--name-only', 'HEAD', self.source_root] > > + p = subprocess.Popen(cmd, stdout=subprocess.PIPE) > > This could be one line. This doesn't really support SVN. Is there a script in the WebKit scripts that can do this for us? No, there isn't. It's very unlikely that someone will make a release without a git mirror, so I don't think it's worth it. I can leave a FIXME just in case someone want to add support for it (I won't) > > Tools/gtk/make-dist.py:129 > > - if not passes_all_rules(file): > > + if not passes_all_rules(file) or self.should_skip_file(file): > > Is this going to do the right thing for files from the build directory? Yes, it actually allows individual files when the source root is not in git (this is the case of files in buildir), assuming that in those cases they have been explicitly added by the manifest. We can try to be more restrictive here, and allow only files that are not in git when they are in the build dir. Created attachment 234750 [details]
Patch
Updated patch. Addressed review comments, and made the rule to skip files more restrictive to only allow files not in git when they are added from build dir.
Comment on attachment 234750 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=234750&action=review > Tools/ChangeLog:9 > + Skip all files in the source tree that are not under the control > + version, except for files add from the build dir like the documentation. Nit: "not under version control" and "files added from..." Committed r171845: <http://trac.webkit.org/changeset/171845> |