The default bug view has changed. See this FAQ.

Remove redundant check-sync-dirs call in js/src/Makefile.in

RESOLVED FIXED in mozilla17

Status

()

Core
Build Config
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: sid0, Assigned: sid0)

Tracking

Trunk
mozilla17
x86_64
Windows 7
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Created attachment 648778 [details] [diff] [review]
patch v1

check-sync-dirs is now run in client.mk (bug 740452), so the call in js/src/Makefile.in is no longer necessary. I've also removed all the configure goop associated with it.

This blocks enabling pymake on Windows because of some funkiness with absolutely configured pymake.
Attachment #648778 - Flags: review?(ted.mielczarek)
Assignee: nobody → sagarwal
Comment on attachment 648778 [details] [diff] [review]
patch v1

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

The only downside here is that people who aren't doing client.mk rebuilds won't notice this until they push. However, it'd still get caught by pushing to try or inbound, and it will fail quickly, so it's not that bad.

::: client.mk
@@ +402,5 @@
>  	          -o \( -name '*.[ao]' -o -name '*.so' \) -type f -print`; \
>  	   build/autoconf/clean-config.sh; \
>  	fi;
>  
> +# Because the SpiderMonkey can be distributed and built independently

"the SpiderMonkey"? I guess that's from the original, but can you fix the grammer?
Attachment #648778 - Flags: review?(ted.mielczarek) → review+
(In reply to Ted Mielczarek [:ted] from comment #1)
> The only downside here is that people who aren't doing client.mk rebuilds
> won't notice this until they push. However, it'd still get caught by pushing
> to try or inbound, and it will fail quickly, so it's not that bad.

As I mentioned on IRC last week, people who aren't doing client.mk builds are unlikely to be running make check anyways, and this check only runs during make check.
(Assignee)

Comment 3

5 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/9dd1dc9801e6
and review comment addressed in
http://hg.mozilla.org/integration/mozilla-inbound/rev/a218e9b11bc5
Status: NEW → ASSIGNED
https://hg.mozilla.org/mozilla-central/rev/9dd1dc9801e6
https://hg.mozilla.org/mozilla-central/rev/a218e9b11bc5
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in before you can comment on or make changes to this bug.