Change .hgignore and .gitignore to ignore Eclipse IDE project files in subdirectories, too

RESOLVED FIXED in mozilla13

Status

()

Core
Build Config
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: till, Assigned: till)

Tracking

Trunk
mozilla13
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

5 years ago
Created attachment 596665 [details] [diff] [review]
patch, V1

In bug 718442, jwatts added Eclipse IDE project files in the root directory to VCS ignore files. I don't really know about other modules but at least for SpiderMonkey it makes sense to create an Eclipse project rooted in js.

The attached patch ignored the files and directory Eclipse creates in all subdirectories. Should that not be desirable, I will add patterns for specific locations instead.
Attachment #596665 - Flags: review?(khuey)
Comment on attachment 596665 [details] [diff] [review]
patch, V1

Review of attachment 596665 [details] [diff] [review]:
-----------------------------------------------------------------

::: .hgignore
@@ +42,5 @@
>  
>  # Ignore the files and directory that Eclipse IDE creates
> +\.project$
> +\.cproject$
> +\.settings$

Hmm, how come there's no trailing '/' on .settings here?
(Assignee)

Comment 2

5 years ago
(In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #1)
> Hmm, how come there's no trailing '/' on .settings here?

Good question. It used to work and still does. I suppose adding the '/' makes sense to prevent weird differences between what git and hg ignore, respectively.
Comment on attachment 596665 [details] [diff] [review]
patch, V1

Review of attachment 596665 [details] [diff] [review]:
-----------------------------------------------------------------

r=me with that change made

Thanks!
Attachment #596665 - Flags: review?(khuey) → review+
(Assignee)

Comment 4

5 years ago
Created attachment 596678 [details] [diff] [review]
patch, V2, carrying r+

Changed .hgignore to only ignore directories called ".settings", not files.

Thanks for the review!
Attachment #596665 - Attachment is obsolete: true
Attachment #596678 - Flags: review+
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
https://hg.mozilla.org/integration/mozilla-inbound/rev/f78a0dea88fc
Status: NEW → ASSIGNED
Keywords: checkin-needed
Target Milestone: --- → mozilla13
https://hg.mozilla.org/mozilla-central/rev/f78a0dea88fc
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.