Closed Bug 415188 Opened 15 years ago Closed 15 years ago

$(CVS_CO_DATE_FLAGS) appears twice in checkout command

Categories

(Firefox Build System :: General, defect)

Product:
Firefox Build System
Component:
General
Platform:
x86
All
Type:
defect
Priority:
Not set
Severity:
trivial

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: dholbert, Assigned: dholbert)

References

Details

Attachments

(1 file)

patch
1.97 KB, patch
ted
: review+
mtschrep
: approval1.9+
Details | Diff | Splinter Review 
When I do a checkout for a specified date, I get something like this:

checkout start: Thu Jan 31 17:11:48 PST 2008
cvs -d :ext:dholbert%cs.stanford.edu@cvs.mozilla.org:/cvsroot -q -z 3  co   -D "11 Nov 2006 04:00 PST" -D "11 Nov 2006 04:00 PST" mozilla/client.mk 

Note that there are two copies of -D "11 Nov 2006 04:00 PST" in that message.


The reason for this is as follows, in client.mk:
* First, CVSCO is defined to already include the date flags:
 498 CVSCO = $(CVS) $(CVS_FLAGS) co $(MOZ_CO_FLAGS) $(if $(MOZ_CO_TAG),-r $(MOZ_CO_TAG)) $(CVS_CO_DATE_FLAGS)

* But then, when we echo and then run our CVS command, we redundantly include the date flags again, after $(CVSCO):
 793  @echo '$(CVSCO) $(CVS_CO_DATE_FLAGS) mozilla/client.mk $(MOZCONFIG_MODULES)'; \
 795  $(CVSCO) $(CVS_CO_DATE_FLAGS) mozilla/client.mk $(MOZCONFIG_MODULES)

URL: http://mxr.mozilla.org/seamonkey/source/client.mk?rev=1.366

It looks like we've been doing this since bug 261232 landed (attachment 166185 [details] [diff] [review]), when the "CVSCO" variable was first defined to include $(CVS_CO_DATE_FLAGS).
Attached patch patch — — Splinter Review 
I think this fixes it.
Comment on attachment 300786 [details] [diff] [review]
patch

Requesting review from ted, since he's reviewed some other recent build-config changes (bug 412923, bug 413253).

Anyone who can review this is welcome to steal the review, though.
Attachment #300786 - Flags: review?(ted.mielczarek)
Assignee: nobody → dholbert
Severity: normal → trivial
Status: NEW → ASSIGNED
OS: Linux → All
Attachment #300786 - Flags: review?(ted.mielczarek) → review+
Comment on attachment 300786 [details] [diff] [review]
patch

Requesting a1.9.

Trivial patch -- just removes a duplicate copy of the date flags in some CVS checkout commands.
Attachment #300786 - Flags: approval1.9?
Attachment #300786 - Flags: approval1.9? → approval1.9+
Patch checked in.
/cvsroot/mozilla/client.mk,v  <--  client.mk
new revision: 1.368; previous revision: 1.367
done
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.