Closed
Bug 5555
Opened 25 years ago
Closed 25 years ago
[PATCH] mozilla/client.mk checkout target broken
Categories
(SeaMonkey :: Build Config, defect, P3)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: hawkfan, Assigned: briano)
Details
Need -d switch when specifying cvsroot on the command line.
= -> := prevents infinite recursion later if we overload the branch.
Index: client.mk
===================================================================
RCS file: /cvsroot/mozilla/client.mk,v
retrieving revision 1.52
diff -u -r1.52 client.mk
--- client.mk 1999/04/26 21:47:47 1.52
+++ client.mk 1999/04/27 15:01:54
@@ -77,11 +77,11 @@
CVS_ROOT_IN_TREE := $(shell cat $(TOPSRCDIR)/CVS/Root 2>/dev/null)
ifneq ($(CVS_ROOT_IN_TREE),)
ifneq ($(CVS_ROOT_IN_TREE),$(CVSROOT))
- CVS_FLAGS := $(CVS_ROOT_IN_TREE)
+ CVS_FLAGS := -d $(CVS_ROOT_IN_TREE)
endif
endif
-CVSCO = cvs $(CVS_FLAGS) co $(CVS_CO_FLAGS)
+CVSCO := cvs $(CVS_FLAGS) co $(CVS_CO_FLAGS)
CVSCO_LOGFILE := $(ROOTDIR)/cvsco.log
####################################
Updated•25 years ago
|
Status: NEW → ASSIGNED
Summary: [PATCH] mozilla/client.mk checkout target broken → [PATCH] mozilla/client.mk checkout target broken
Comment 1•25 years ago
|
||
Thanks for the patch.
Oops, I forgot the '-d' flag. I'll add that.
As for $(CVSCO), it needs to have "recursive expansion" so that client.mk can
checkout mozconfig2defs.sh and find-mozconfig.sh before it reads the client.mk
options. (Chicken and egg problem.) To avoid the infinite loop, I'll change the
way the branch and date flags get added.
Index: client.mk
===================================================================
RCS file: /cvsroot/mozilla/client.mk,v
retrieving revision 1.52
diff -r1.52 client.mk
39d38
< # also,
80c79
< CVS_FLAGS := $(CVS_ROOT_IN_TREE)
---
> CVS_FLAGS := -d $(CVS_ROOT_IN_TREE)
121c120
< CVSCO := $(CVSCO) -r $(MOZ_CO_BRANCH)
---
> CVS_CO_FLAGS := $(CVS_CO_FLAGS) -r $(MOZ_CO_BRANCH)
125c124
< CVSCO := $(CVSCO) -D "$(MOZ_CO_DATE)"
---
> CVS_CO_FLAGS := $(CVS_CO_FLAGS) -D "$(MOZ_CO_DATE)"
Updated•25 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Comment 2•25 years ago
|
||
I checked in my patch.
Comment 3•25 years ago
|
||
In case you are interested, here is a handy shell function I use to switch
between CVS roots.
CVSROOT_LIST="
gila :pserver:$USER%netscape.com@gila.mozilla.org:/cvsroot
mozilla :pserver:$USER%netscape.com@cvs.mozilla.org:/cvsroot
gnome :pserver:anonymous@anoncvs.gnome.org:/cvs/gnome
"
function cvsroot()
{
echo $CVSROOT_LIST | perl -n -e 'chomp;
@root_list = split / /;
%rootname = reverse @root_list;
print "\n";
for ($ii=1;
$name = shift @root_list and $root = shift @root_list;
$ii++) {
printf "%2d. %-10s %s\n", $ii, $name, $root;
}
print "\nPlease select root [".$rootname{$ENV{CVSROOT}}."]: ";'
read _root_num
echo
_root_cmd=`echo $CVSROOT_LIST | perl -n -e 'chomp;$root_num="'"$_root_num"'";
@root_list = split / /;
print "CVSROOT=$ENV{CVSROOT}" if $root_num eq "";
print "CVSROOT=@root_list[$root_num*2-1]" if $root_num ne "";'`
echo $_root_cmd
eval $_root_cmd
}
Updated•20 years ago
|
Product: Browser → Seamonkey
(In reply to comment #4)
> ttt
sdfsdffsdfsfdsdfsdsdsfsfsfdsfsdfsdfsdfsdfsdsfdsdsdsdfsddsdsf
You need to log in
before you can comment on or make changes to this bug.
Description
•