Issue 1404 - several CVS commands return the long-version repository path
Summary: several CVS commands return the long-version repository path
Status: CLOSED FIXED
Alias: None
Product: Infrastructure
Classification: Infrastructure
Component: Upgrade (show other issues)
Version: current
Hardware: PC Other OS
: P1 (highest) Trivial (vote)
Target Milestone: ---
Assignee: Unknown
QA Contact: issues@www
URL:
Keywords:
Depends on:
Blocks: 2177
  Show dependency tree
 
Reported: 2001-08-03 13:36 UTC by Martin Hollmichel
Modified: 2013-08-07 15:24 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---


Attachments

Note You need to log in before you can comment on or make changes to this issue.
Description Martin Hollmichel 2001-08-03 13:36:58 UTC
it comers again /usr/local/tigris/data/helm/cvs/repository instead of /cvs
Comment 1 Martin Hollmichel 2001-08-03 14:43:31 UTC
seems to be fixed now
Comment 2 Martin Hollmichel 2001-08-03 17:59:48 UTC
cvs commit gives still /usr/local/tigris.... path when checking in
Comment 3 Unknown 2001-08-04 00:04:54 UTC
Is the echoed commit path an issue for any of your automated 
processes? My understanding is that the log path was critical. Is 
this issue breaking anything, or is it just annoying?

If just annoying, I'd like to downgrade to P3. Let me know 
otherwise and I will arrange a fix quickly. (I am trying to prioritize).
Comment 4 Unknown 2001-08-04 00:42:09 UTC
No, it's not only annoying.
There are tools which collect the file and rev of a just committed 
file.
Comment 5 Unknown 2001-08-06 16:05:24 UTC
Will begin resolution this morning. Kat, let's get with thom to 
patch this as he did with the log.
Comment 6 Martin Hollmichel 2001-08-06 16:12:26 UTC
I saw that diff, update and other commands still display 
the /usr/local/...  path.

I think live will be much easier for all if cvs provides overall 
consistant paths to the repository for all cvs commands.
Comment 7 Unknown 2001-08-06 16:55:01 UTC
New summary to better reflect the situation.
Comment 8 Unknown 2001-08-06 23:51:55 UTC
Issue pcn5313 has been created internally for tracking.
Comment 9 Unknown 2001-08-07 01:43:01 UTC
The underlying cause of this problem is that the application 
uses a symlink between the 'symbolic' CVS directory (/cvs) and 
the actual location of the repository: the fully qualified Unix path 
you guys are seeing in CVS messages.

We are going to test a long-term fix to this problem over the next 
two days. If it is successful, we will need to stop the pserver on 
Wednesday or Thrusday for a period of less than one hour to 
update the OOo site.
Comment 10 Unknown 2001-08-09 02:08:27 UTC
Our long-term fix test did not work as expected.

Here is our new, more certain proposal:

Embedded within the CVS application code is symlink 
information. We propose to strip all this stuff out and put it into 
the CVS configuration .pl file.

This will solve the problem now and will make it a nonissue for 
any future upgrades.

The effort will take up to five days to code, stage and implement. 
Site downtime will be a matter of minutes at go-live.

I have already started the process rolling.
Comment 11 Unknown 2001-08-29 23:01:24 UTC
Apologies - the QA queue has caused me to push this one out 
again. I believe it will approximately two weeks before we can 
promote this to the OOo production server.

As we get closer, I will firm up the implementation date.
Comment 12 Unknown 2001-10-18 23:46:54 UTC
Hi,

A fix for this issue has been verified by QA in version 1.1 of
SourceCast and will be present in the upgrade to this version. 

Thank you
Kat

Comment 13 stx123 2001-11-06 19:02:46 UTC
reassigned
Comment 14 Unknown 2001-11-07 21:21:31 UTC
accepting
Comment 15 Unknown 2001-11-13 03:52:50 UTC
Tested commit on staging server, the short path is returned:

[kat@localhost www]$ cvs -d :pserver:kat@localhost:/cvs commit
about.html 
Checking in about.html;
/cvs/www/www/about.html,v  <--  about.html

We will be testing other commands, please also provide feedback on any
you find are still returning the long path.

Thank you
Kat
Comment 16 Martin Hollmichel 2001-11-13 17:22:56 UTC
on the staging box:

the cvs log output looks fine, but after "cvs co" the CVS/Repository 
file does not contain the full path, e.g. it should 
contain /cvs/util/sot/inc, but the content is util/sot/inc.
Comment 17 Unknown 2001-11-14 00:18:14 UTC
Hi,

The solution in place currently, and the one available for the
upgrade, is a workaround only. The long term fix to virtualize the cvs
path is much more involved and is being tracked separately as the
internal issue pcn5949 targeted for a later release.

We are investigating the change in the CVS/Repository entry through
issue pcn6508 internally.

Thank you
Kat


Comment 18 Martin Hollmichel 2001-11-14 20:16:56 UTC
I noticed that the CVS/Repository files are fixed in the meantime. I 
assume that something has changed on your side ?!?
Comment 19 Unknown 2001-11-15 01:11:02 UTC
Hi Martin,

Nothing has changed on our side that I am aware of. The Repository
file when i co util/sot/inc (and other modules) from the staging
server still shows the module without the preceeding /cvs as you noted
previously.
Which module did you try this with and find different results?

Thank you
Kat
Comment 20 Martin Hollmichel 2001-11-15 11:41:48 UTC
OK, here some clarification:

there are two valid possiblities for the CVS/Repository file:
1. the absolute path (e.g. /cvs/util/sot)
2. the relative path (e.g. util/sot)

If you mix up both kind of entries, things will not work.

if a cvs client 1.10.x is used for checkout, absolute path will 
generated, using cvs client 1.11.x is generating relative paths.

so, if somebody mix up these kind of clients (like I did), he will 
get into trouble.

the same behaviour is already in SC 1.0.8
Comment 21 stx123 2001-11-15 12:10:58 UTC
As the original problem (/usr/local/tigris/data/helm/cvs/...) paths in
cvs command is fixed, we can close this issue. If we find new CVS
related issues we will open new bugs.

Comment 22 Martin Hollmichel 2002-05-23 11:46:06 UTC
closed