mozilla-central .hgignore file is incomplete

RESOLVED FIXED

Status

defect
RESOLVED FIXED
12 years ago
a year ago

People

(Reporter: jorendorff, Assigned: jorendorff)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

Assignee

Description

12 years ago
"hg status" should skip over:

  * Emacs ~ files
  * directories pulled from CVS

The way to do this is to add the right regexps to .hgignore.
Attachment #272044 - Flags: review?(ted.mielczarek)
Assignee

Comment 1

12 years ago
(The previous patch deleted regexps for CVS directories.  Mistake.)
Attachment #272044 - Attachment is obsolete: true
Attachment #272044 - Flags: review?(ted.mielczarek)
Assignee

Updated

12 years ago
Attachment #272046 - Flags: review?(ted.mielczarek)
Assignee

Comment 2

12 years ago
Comment on attachment 272044 [details] [diff] [review]
.hgignore patch

OK, after a cup of coffee I think the first patch is the correct one.  CVS directories shouldn't show up in your Mercurial repository, with the exception of the cvs-trunk-mirror repository.  So I'll check the first patch into mozilla-central (and not cvs-trunk-mirror).
Attachment #272044 - Attachment is obsolete: false
Assignee

Updated

12 years ago
Attachment #272046 - Flags: review?(ted.mielczarek) → review-

Comment 3

12 years ago
CVS directories *do* show up when we're doing import from CVS, and so .hgignore must continue to ignore them. Please don't change this, as it's important ;-)
Assignee

Comment 4

12 years ago
Pushed changeset 899ca1f39f07 to hg.mozilla.org/mozilla-central .
Status: NEW → RESOLVED
Last Resolved: 12 years ago
OS: Mac OS X → All
Hardware: PC → All
Resolution: --- → FIXED
Assignee

Comment 5

12 years ago
bs: Just talked with luser about this on IRC...

I think we should only be doing import from CVS in cvs-trunk-mirror.  If a CVS directory showed up in your mozilla-central or actionmonkey repository, it would mean something weird has happened somewhere, and you might as well know about it.

So this change would be bad in cvs-trunk-mirror, but it makes sense in mozilla-central (where it's committed right now).

Or so the theory goes.  Let me know.

Updated

a year ago
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.