Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 296541

Summary: Eclipse e4 repositories should be mirrored by Git
Product: Community Reporter: Lars Vogel <Lars.Vogel>
Component: ServersAssignee: Eclipse Webmaster <webmaster>
Status: RESOLVED WORKSFORME QA Contact:
Severity: normal    
Priority: P3 CC: bernhard.merkle, bokowski, bokowski, bsd, contact, eduard.bartsch, Lars.Vogel, pwebster, simon_kaegi, Szymon.Brandys
Version: unspecifiedKeywords: helpwanted
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Bug Depends on: 345479    
Bug Blocks:    
Attachments:
Description Flags
options for e4 cvs repository for cvs2git
none
broken_komma_v_cvs_files_found_by_cvs2git
none
patched ,v files for e4.ressources project
none
Diff log none

Description Lars Vogel CLA 2009-11-30 23:24:19 EST
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-GB; rv:1.9.1.5) Gecko/20091102 Firefox/3.5.5
Build Identifier: 

Hi,

I believe the E4 cvs repository is currently not available as Git mirror. At least I don't see it at: http://dev.eclipse.org/git/

Can you please change this?

Best regards, Lars

Reproducible: Always
Comment 1 Denis Roy CLA 2009-12-01 06:44:17 EST
Moving to Community/Server.  I started the import hours ago, and the index should be generated soon.
Comment 2 Denis Roy CLA 2009-12-02 09:02:28 EST
I'm mirroring all the org.eclipse.* directories in this CVS location:
http://dev.eclipse.org/viewcvs/index.cgi/e4/

e4 is now listed on the git page:
http://dev.eclipse.org/git/

Closing fixed.  Reopen if I have missed something.
Comment 3 Lars Vogel CLA 2009-12-04 00:34:41 EST
Hi Denis,

If it try to clone the repository I receive the message that the directory is empty:

Here my command and the error message: 

vogella@ubuntu:~/eclipsesource$ git clone git://dev.eclipse.org/org.eclipse.e4/org.eclipse.e4.ui.git
Initialized empty Git repository in /home/vogella/eclipsesource/org.eclipse.e4.ui/.git/
warning: You appear to have cloned an empty repository.

I tried several repos but the same for all of them. 

Can you please re-check?

Best regards, Lars
Comment 4 Denis Roy CLA 2009-12-04 15:47:31 EST
/usr/local/libexec/git-core/git-cvsimport -i -p -x -d :local:/home/data/cvs/eclipse e4/org.eclipse.e4.resources -C /home/data/git/org.eclipse.e4
Initialized empty Git repository in /home/data/git/org.eclipse.e4/org.eclipse.e4.resources.git/
revision 1.4 of file bundles/org.eclipse.ui.ide/icons/full/etool16/undo_edit.gif is tagged but not present
revision 1.4 of file bundles/org.eclipse.ui.ide/icons/full/etool16/saveas_edit.gif is tagged but not present
[snip]
git-cvsimport: fatal: cvsps reported error

This is happening on two e4 repos.

I'll see if I can make sense of that amazingly cryptic error.
Comment 5 Boris Bokowski CLA 2009-12-04 16:16:26 EST
(In reply to comment #4)
> revision 1.4 of file
> bundles/org.eclipse.ui.ide/icons/full/etool16/saveas_edit.gif is tagged but not
> present

Have a look at this file:
/cvsroot/eclipse/e4/org.eclipse.e4.resources/bundles/org.eclipse.ui.ide/icons/full/etool16/Attic/saveas_edit.gif,v

It contains a line:
  v20040316:1.4;

Which is bogus because the revision history says that there were revisions 1.1 and 1.2, and then a "deletion record" with revision 1.3.

I would suggest deleting the offending line in the files that cause these problems.

The same error should happen when converting /cvsroot/eclipse/org.eclipse.ui.ide to Git - that's where the "org.eclipse.ui.ide" was copied from when it was "hard forked" for e4.
Comment 6 Denis Roy CLA 2009-12-04 16:47:26 EST
Boris, that type of error happens in a lot of our CVS repos, but it doesn't prevent the import from functioning.  I'll have to dig deeper.
Comment 7 Denis Roy CLA 2010-01-08 10:48:30 EST
*** Bug 299135 has been marked as a duplicate of this bug. ***
Comment 8 Boris Bokowski CLA 2010-01-25 14:35:25 EST
Hi Denis,

Any progress on this? Since there are EGit builds available now, I wanted to try it out on e4 code I am working on but then hit this problem.
Comment 9 Boris Bokowski CLA 2010-01-25 14:37:04 EST
(In reply to comment #5)
> The same error should happen when converting
> /cvsroot/eclipse/org.eclipse.ui.ide to Git - that's where the
> "org.eclipse.ui.ide" was copied from when it was "hard forked" for e4.

Just as a data point - trying to import git://dev.eclipse.org/org.eclipse.ui/org.eclipse.ui.ide.git does not work.
Comment 10 Boris Bokowski CLA 2010-01-25 14:44:26 EST
(In reply to comment #5)
> I would suggest deleting the offending line in the files that cause these
> problems.

Sorry for the third bug spam...

Since the offending lines listed in comment 5 don't make sense at all, not even when just using CVS, wouldn't it be worth trying to fix the ",v" files, followed by a new git import attempt?

I would volunteer to change the two ",v" files, including taking full responsibility if it breaks anything on the CVS side. :-)
Comment 11 Denis Roy CLA 2010-01-26 15:10:55 EST
> Since the offending lines listed in comment 5 don't make sense at all, not even
> when just using CVS, wouldn't it be worth trying to fix the ",v" files,
> followed by a new git import attempt?

I'm not convinced the lines are 'offending' at all.  cvsps is segfaulting on a specific set of e4 repositories.  I'll try some experimentation offline so that I don't bamboozle the live repo.
Comment 12 Lars Vogel CLA 2010-02-28 14:12:52 EST
Any news on the investigation?

Sorry to ask again but it would be really nice to have the possibility to clone the e4 repository. 

I'm mostly caught behind a firewall with no access to the cvs repository and in the evening my internet connection is slow and unstable. It would be nice to sync during the day via http git and then be able to work on / look at the code in the evening.
Comment 13 Bernhard Merkle CLA 2010-03-12 14:30:09 EST
same for me. company firewall blocks CVS and workarounds.
Comment 14 Boris Bokowski CLA 2010-04-22 14:48:31 EDT
Denis, no updates from you since January. Is there anything we can do to help? For example, what are the command line options (or script) you are using to convert from CVS to Git? If you made a CVS tarball of /cvsroot/eclipse available, the community could try to track down the cause of the problem. The wiki page at [1] points to a mostly empty web page.

[1] http://wiki.eclipse.org/index.php/CVS_Howto#CVS_tarball_snapshots
Comment 15 Bernhard Merkle CLA 2010-04-22 15:07:05 EDT
Boris, if you tar/gzip me the original CVS e4 repo (means all e4 ,v files) I can give it a try. I have done several CVS2SVN and CVS2Git conversions so maybe i can help.

thanks,
Berni.
Comment 16 Denis Roy CLA 2010-04-22 15:12:50 EDT
(In reply to comment #14)
> Denis, no updates from you since January. Is there anything we can do to help?
> For example, what are the command line options (or script) you are using to
> convert from CVS to Git?

I'm using git-cvsimport:

git-cvsimport -i -p -x -d :local:/home/data2/cvs/eclipse e4/ -C ./e4/

Not that although I am using :local: anyone can use :pserver: and run this from anywhere.

You don't need to download tarballs.

> If you made a CVS tarball of /cvsroot/eclipse
> available, the community could try to track down the cause of the problem. The
> wiki page at [1] points to a mostly empty web page.
> 
> [1] http://wiki.eclipse.org/index.php/CVS_Howto#CVS_tarball_snapshots

I fixed the blank page but ... don't download a 5G tarball just for this.  Just use pserver.
Comment 17 Denis Roy CLA 2010-04-22 15:13:52 EDT
> Denis, no updates from you since January.

Sorry, I just can't do everything.
Comment 18 Bernhard Merkle CLA 2010-04-22 15:27:46 EDT
(In reply to comment #16)
>> You don't need to download tarballs.
> 
> > If you made a CVS tarball of /cvsroot/eclipse
> > available, the community could try to track down the cause of the problem. The
> > wiki page at [1] points to a mostly empty web page.
> > 
> > [1] http://wiki.eclipse.org/index.php/CVS_Howto#CVS_tarball_snapshots
> 
> I fixed the blank page but ... don't download a 5G tarball just for this.  Just
> use pserver.

the problem with pserver is that it will take quite some time from germany to do this. I have not tested this but I could imagine.

also, as I understand the problem there is a issue with the current CVS ,v file in one or two cases which causes problems for the conversion, so I need really the ,v file to fix this there. (pserver does not work in this case)

regarding the tar bar. I do not need the whole 5GB (I do not want this actually). Just tar me up the e4 directory and that should be really small i think.
I only need the e4 part for this IIRC...

thanks,
Berni.
Comment 19 Bernhard Merkle CLA 2010-04-22 15:28:38 EDT
(In reply to comment #17)
> > Denis, no updates from you since January.
> 
> Sorry, I just can't do everything.

Thats why I offer help. tar me up the e4 directory and put is somewhere i can download. :-)
Comment 21 Boris Bokowski CLA 2010-04-22 17:02:12 EDT
I imagine Bernhard will also need the stuff under /cvsroot/eclipse/CVSROOT for the CVS metadata. Denis, could you put that in another tarball for Bernhard to download? Thanks!
Comment 22 Bernhard Merkle CLA 2010-04-22 17:25:36 EDT
question: 
- will we make a direct switch (after some testing of course ;-) an then proceed with git and retire cvs for the e4 stuff or 
- stay for some time with cvs and only mirror into git

?
Comment 23 Bernhard Merkle CLA 2010-04-22 17:27:39 EDT
for testing the result i need a few relevant labels which i then checkout and compare the cvs and git stuff. Can you tell me the interesting candidates :-) ?
Comment 24 Lars Vogel CLA 2010-04-22 17:36:04 EDT
Not sure if you mean this but plugin "org.eclipse.e4.ui.model.workbench" is pretty important.
Comment 25 Bernhard Merkle CLA 2010-04-22 18:31:22 EDT
(In reply to comment #21)
> I imagine Bernhard will also need the stuff under /cvsroot/eclipse/CVSROOT for
> the CVS metadata. Denis, could you put that in another tarball for Bernhard to
> download? Thanks!

Thanks but I do not need CVSROOT. The converter I use does not need that information and actually all information needed is in the ,v files.

But what I can use is a mapping from the cvs user names to e-mail adresses if you plan to use the notation "J. Random <jrandom@example.com>" in git.

The default used at the moment is "cvsauthor <cvsauthor>"

I suggest that you send me this information via private e-mail :-) if you want to use the e-mail adresses.
Comment 26 Bernhard Merkle CLA 2010-04-23 04:33:47 EDT
Boris, Denis the conversion for the complete 4 repository is done. :-)

I tested HEAD and 3 tags and the output of CVS and git is identical.

Please tell me where I can upload the converted stuff.

I will also write up a summary how to do the conversion, so you can setup it at your site. Please also see #comment25 for the login/email stuff i could need.
Comment 27 Bernhard Merkle CLA 2010-04-23 04:34:45 EDT
(In reply to comment #26)
> Boris, Denis the conversion for the complete 4 repository is done. :-)

I meant "complete e4 repository" of course, sorry
Comment 28 Denis Roy CLA 2010-04-23 08:47:52 EDT
Thanks for your efforts.

Just to be clear, I would have appreciated finding a solution to the current issue, where the git-cvsimport process used to mirror, on a regular basis, is segfaulting with the cvsps call.

Unless I misunderstand, a one-off conversion is not what is needed now, unless e4 is ready to move off CVS entirely.  Even then, such a conversion would have to be performed by either myself or a committer using data and tools they know and trust.  I hope you understand -- I don't know what is inside your "converted stuff"  :-)

If we can figure out what is preventing the repo from being mirrored correctly, that would be awesome.
Comment 29 Bernhard Merkle CLA 2010-04-23 09:57:10 EDT
(In reply to comment #28)
> Thanks for your efforts.
> 
> Just to be clear, I would have appreciated finding a solution to the current
> issue, where the git-cvsimport process used to mirror, on a regular basis, is
> segfaulting with the cvsps call.
> 
> Unless I misunderstand, a one-off conversion is not what is needed now, unless
> e4 is ready to move off CVS entirely.  Even then, such a conversion would have
> to be performed by either myself or a committer using data and tools they know
> and trust.  I hope you understand -- I don't know what is inside your
> "converted stuff"  :-)
> 
> If we can figure out what is preventing the repo from being mirrored correctly,
> that would be awesome.

there are no secrets :-) I use cvs2svn / better cvs2git to perform the conversion. This is _much_ more reliable than git-cvsimport which uses cvsps, which is essentially dead as i understand. 
also cvs2git gives you the reasons why cvsps segfaults and cvs2git is much faster than git-cvsimport.

The workflow is a one-time cvs2git conversion which i now succeeded 
+ a following mirroring/sync of modifications in cvs to the git repo 
(e.g. via git-cvsimport)

I will write up detailed summary how _you_ can do this also so you do not have to trust my converted git repo. However it would be great it some of the e4 commiters can do just a test with my converted stuff, just as a test.
Do you see any problems with that ? Is there a dir where i can upload the converted stuff ?

thanks,
Berni.
Comment 30 Boris Bokowski CLA 2010-04-23 12:54:03 EDT
(In reply to comment #29)
> Do you see any problems with that ? Is there a dir where i can upload the
> converted stuff ?

GitHub?

Apologies if we haven't been clear enough - we cannot afford the churn that would be caused by moving e4 to Git right now. But I am all for having a Git mirror that is continually synced with CVS.

My idea was that we could help Denis figure out how to improve the current CVS->Git mirroring that he set up.
Comment 31 Lars Vogel CLA 2010-04-23 12:58:13 EDT
@Denis: is using cvs2git  an option for you

@Bernhard: Did you get an error message regarding the segfault which you can share with Denis so that he can fix this issue?

Quote:
-------------
cvs2git also gives you the reasons why cvsps segfaults and cvs2git is much
faster than git-cvsimport.
---------
Comment 32 Bernhard Merkle CLA 2010-04-23 13:00:40 EDT
(In reply to comment #30)

> My idea was that we could help Denis figure out how to improve the current
> CVS->Git mirroring that he set up.

yes thats a good idea. I will post the instructions how to convert and mirror the current CVS e4 repo with cvs2git. I tested the converted stuff and it is ok.
This is much more stable than using git cvs-import :-)
Comment 33 Bernhard Merkle CLA 2010-04-23 13:03:52 EDT
(In reply to comment #31)
> @Denis: is using cvs2git  an option for you
> 
> @Bernhard: Did you get an error message regarding the segfault which you can
> share with Denis so that he can fix this issue?

the problem is that some of the cvs files are broken and cvsps segfaults on them.
cvs2git is more fault tolerant and outputs the reason. We can fix this in the cvs ,v files, however cvs2git is so much faster and faulttolerant that it IS the way to go.


> Quote:
> -------------
> cvs2git also gives you the reasons why cvsps segfaults and cvs2git is much
> faster than git-cvsimport.
> ---------

yes, see comment #29 :-)
Comment 34 Boris Bokowski CLA 2010-04-23 13:56:07 EDT
(In reply to comment #33)
> the problem is that some of the cvs files are broken and cvsps segfaults on
> them.

How many files are broken?

> cvs2git is more fault tolerant and outputs the reason. We can fix this in the
> cvs ,v files, however cvs2git is so much faster and faulttolerant that it IS
> the way to go.

You are probably right, but the current set up (using cvsps) seems to work for most other projects. It might be easiest (for now) to fix the CVS ,v files and look at changing the way the import works later.
Comment 35 Bernhard Merkle CLA 2010-04-23 14:08:48 EDT
ok, i used http://cvs2svn.tigris.org/ it has not only a svn but also git/hg/bzr output. I used the current trunk version in addition to cvs2svn 2.3.0.

- install cvs2svn 2.3.0 (just to get a clean cvs2svn environment)
sudo apt-get install cvs2svn

- get the current trunk version of cvs2svn (has better git support)
http://cvs2svn.tigris.org/svn/cvs2svn/trunk/
run the tests .e.g in cvs2svn-trunk ./run-tests.py

- in cvs2svntrunk ./cvs2git --options=e4-cvs2git.options
now conversion runs about 40min or so... 
in the first stage it also gives infos about the broken cvs ,v files

-import into git via fast import
#
git init
cat cvs2svn-tmp/git-blob.dat cvs2svn-tmp/git-dump.dat | git fast-import

- then perform tests, like checkout head from cvs and master from git.
compare the two working copies with a recursive diff gave no differences for me.

please see also http://cvs2svn.tigris.org/cvs2git.html

e4-cvs2git.options file is attached. I changed the dirs and encoding.
Comment 36 Bernhard Merkle CLA 2010-04-23 14:12:27 EDT
Created attachment 165945 [details]
options for e4 cvs repository for cvs2git
Comment 37 Bernhard Merkle CLA 2010-04-23 14:27:45 EDT
(In reply to comment #34)
> (In reply to comment #33)
> > the problem is that some of the cvs files are broken and cvsps segfaults on
> > them.
> 
> How many files are broken?

38

> 
> > cvs2git is more fault tolerant and outputs the reason. We can fix this in the
> > cvs ,v files, however cvs2git is so much faster and faulttolerant that it IS
> > the way to go.
> 
> You are probably right, but the current set up (using cvsps) seems to work for
> most other projects. It might be easiest (for now) to fix the CVS ,v files and
> look at changing the way the import works later.

okay attached you can find the list of broken ,v files, found by cvs2git.
If you fix them (usually delete that invalid tag/label) cvsps will work (i hope)

I would (recommend to) switch to cvs2svn/git in the longer term :-)
At work I converted a large amount of our repos from CVS/PVCS/<insert other VCS> with cvs2svn.
Comment 38 Bernhard Merkle CLA 2010-04-23 14:28:21 EDT
Created attachment 165948 [details]
broken_komma_v_cvs_files_found_by_cvs2git
Comment 39 Boris Bokowski CLA 2010-04-23 15:25:55 EDT
(In reply to comment #37)
> okay attached you can find the list of broken ,v files, found by cvs2git.
> If you fix them (usually delete that invalid tag/label) cvsps will work (i
> hope)

Done.

Denis, can you try to run the Git import for e4 to confirm that it no longer segfaults? (Or will the script run anyway overnight?)
Comment 40 Denis Roy CLA 2010-04-23 15:40:17 EDT
The script runs every day at 1pm ET.  I'm not sure if it has processed e4 yet (doubt it) but it will run automatically.
Comment 41 Bernhard Merkle CLA 2010-04-25 08:42:11 EDT
I tried to test but dev.eclipse.org seems to be down since Sat evening...

Ping dev.eclipse.org [206.191.52.50] mit 32 Bytes Daten:

Zeitüberschreitung der Anforderung.

Ping-Statistik für 206.191.52.50:
    Pakete: Gesendet = 1, Empfangen = 0, Verloren = 1 (100% Verlust),
Comment 42 Denis Roy CLA 2010-04-26 09:09:57 EDT
You'll never get a ping reply from dev.eclipse.org.
Comment 43 Bernhard Merkle CLA 2010-04-26 16:36:59 EDT
(In reply to comment #40)
> The script runs every day at 1pm ET.  I'm not sure if it has processed e4 yet
> (doubt it) but it will run automatically.

testing with the "ui" and "resources" project still fails...

did the script run ?

C:\_test>C:\Programme\Git\bin\git clone git://dev.eclipse.org/org.eclipse.e4/org.eclipse.e4.ui.git
Initialized empty Git repository in C:/_test/org.eclipse.e4.ui/.git/
fatal: The remote end hung up unexpectedly

C:\_test>C:\Programme\Git\bin\git clone git://dev.eclipse.org/org.eclipse.e4/org.eclipse.e4.resources.git
Initialized empty Git repository in C:/_test/org.eclipse.e4.resources/.git/
fatal: The remote end hung up unexpectedly
Comment 44 Denis Roy CLA 2010-04-26 17:11:14 EDT
This morning I completely deleted all the e4 Git mirrors and re-started the import.  Some had succeeded, but I'm guessing others have not.  I'll provide some more details tomorrow since I'm not at the computer that did the import.
Comment 45 Boris Bokowski CLA 2010-04-27 00:11:30 EDT
(In reply to comment #43)
> testing with the "ui" and "resources" project still fails...

It works for me now with "ui" but not with "resources". It looks like we have at least some progress. Denis, do you have a log file from the last import? Or could you create an updated tar for Bernhard to check for any files that I might have missed?
Comment 46 Bernhard Merkle CLA 2010-04-27 04:16:51 EDT
(In reply to comment #45)
> (In reply to comment #43)
> > testing with the "ui" and "resources" project still fails...
> 
> It works for me now with "ui" but not with "resources". It looks like we have
> at least some progress. Denis, do you have a log file from the last import? Or
> could you create an updated tar for Bernhard to check for any files that I
> might have missed?

we can do this again. Please tar me up again the e4 rep and I will have a look at this. I will additionally test with cvsps this time and locate the crash.

However in the longer term, please consider #comment 30 and switching to cvs2svn resp. cvs2git. http://cvs2svn.tigris.org/cvs2git.html
It is actively maintained, faster and more reliable than cvsps.
You said that is is working with cvsps, and I believe that, but i am unsure if the conversion really took over all tags/branches etc. in the right way.
Have you tested this ? (compare the converted output). ?
There are numerous bug reports for cvsps that it that problems with various scenarios in branches and tags...

Also #comment 35 includes a complete description how to do this with cvs2svn/git.

maybe the following links are also worth to consider

[1] http://www.gelato.unsw.edu.au/archives/git/0506/5389.html
[2] http://www.oak.homeunix.org/~marcel/blog/2009/06/03/tracking-cvs-with-git-using-cvs2git
Comment 47 Denis Roy CLA 2010-04-27 10:22:46 EDT
> Denis, do you have a log file from the last import?

These are the final lines from the 'resources' import:

cvs_direct: rlog: read E cvs rlog: Logging e4/org.eclipse.e4.resources/bundles/org.eclipse.ui.ide/icons/full/elcl16
cvs rlog: Logging e4/org.eclipse.e4.resources/bundles/org.eclipse.ui.ide/icons/full/elcl16
state: 0 read line:=============================================================================

cvs_direct: rlog: read M 
state: 0 read line:

cvs_direct: rlog: read M RCS file: /home/data2/cvs/eclipse/e4/org.eclipse.e4.resources/bundles/org.eclipse.ui.ide/icons/full/elcl16/Attic/addbkmrk_tsk.gif,v
state: 0 read line:RCS file: /home/data2/cvs/eclipse/e4/org.eclipse.e4.resources/bundles/org.eclipse.ui.ide/icons/full/elcl16/Attic/addbkmrk_tsk.gif,v

stripped filename bundles/org.eclipse.ui.ide/icons/full/elcl16/addbkmrk_tsk.gif
new file: bundles/org.eclipse.ui.ide/icons/full/elcl16/addbkmrk_tsk.gif
cvs_direct: rlog: read M head: 1.3
state: 2 read line:head: 1.3

cvs_direct: rlog: read M branch:
state: 2 read line:branch:

cvs_direct: rlog: read M locks:
state: 2 read line:locks:

cvs_direct: rlog: read M access list:
state: 2 read line:access list:

cvs_direct: rlog: read E Terminated with fatal signal 11
Terminated with fatal signal 11
state: 2 read line:access list:

cvs_direct: rlog: read error  
state: 2 read line:access list:
Comment 48 Denis Roy CLA 2010-04-27 10:24:45 EDT
(In reply to comment #46)
> However in the longer term, please consider #comment 30 and switching to
> cvs2svn resp. 

Consider it considered.
Comment 49 Bernhard Merkle CLA 2010-04-29 19:38:24 EDT
(In reply to comment #45)
> (In reply to comment #43)
> > testing with the "ui" and "resources" project still fails...
> 
> It works for me now with "ui" but not with "resources". It looks like we have
> at least some progress. Denis, do you have a log file from the last import? Or
> could you create an updated tar for Bernhard to check for any files that I
> might have missed?

Boris could it be that not all ,v have been patched correctly ?
I did this now also in my patched e4 CVS repo and got no error with the approach "git cvs-import" which you are using.
There are some variation with that invalid labels (e.g. rev number) so I you have patched this with sed maybe not all patterns were replaced ?

anyway, appended i a tar.gz of the relevant e4.resources part. this should fix the crash.

I did not produce a patch file but just look for ~ files in the tar.gz and you will see what I have changed. 

the "git cvs-import" rans fine now (but it is still running and takes ages)...
Comment 50 Bernhard Merkle CLA 2010-04-29 19:39:32 EDT
Created attachment 166585 [details]
patched ,v files for e4.ressources project

patched ,v files for e4.ressources project
Comment 51 Denis Roy CLA 2010-05-05 09:03:17 EDT
I'm looking into cvs2svn today...
Comment 52 Lars Vogel CLA 2010-07-06 03:33:07 EDT
@Bernhard: As far as I can tell you did the conversion from the tar file. How would I run this conversion directly from cvs?

Lets say I checkout on plugin from cvs, e.g. 

cvs -d ':pserver:anonymous@dev.eclipse.org/cvsroot/eclipse' co e4/org.eclipse.e4.ui/bundles/org.eclipse.e4.ui.workbench3

What would be the next step to convert the repo to git? Perhaps I can help cleaning up the errors.
Comment 53 Brian de Alwis CLA 2010-07-07 08:02:47 EDT
So I believe this has been fixed — at least I am now able to pull from the repository, and it appears to be up-to-date with recent changes.
Comment 54 Bernhard Merkle CLA 2010-07-08 02:39:33 EDT
(In reply to comment #52)
> @Bernhard: As far as I can tell you did the conversion from the tar file. How
> would I run this conversion directly from cvs?
> 
> Lets say I checkout on plugin from cvs, e.g. 
> 
> cvs -d ':pserver:anonymous@dev.eclipse.org/cvsroot/eclipse' co
> e4/org.eclipse.e4.ui/bundles/org.eclipse.e4.ui.workbench3
> 
> What would be the next step to convert the repo to git? Perhaps I can help
> cleaning up the errors.

@Lars, please see comment#50. there were some broken ,v CVS files and i fixed them. I think Dennis has applied them to the CVS repo and now the cvs2git mirror process runs, so it seems to be fixed now.
Comment 55 Denis Roy CLA 2010-07-09 11:51:01 EDT
(In reply to comment #53)
> So I believe this has been fixed — at least I am now able to pull from the
> repository, and it appears to be up-to-date with recent changes.

I never got a chance to look into cvs2svn, but I did upgrade Git to the latest version... I think that has solved numerous problems.

Closing as FIXED since everything seems to be working ok.  Please reopen if I've missed something.
Comment 56 Lars Vogel CLA 2010-07-13 09:03:43 EDT
Seems to be working fine. Thanks to all. Special thanks Denis and Bernhard for spending time on tracking this down.
Comment 57 Eduard Bartsch CLA 2010-07-28 05:38:10 EDT
I am reopening the bug because the repository org.eclipse.e4.resources.git is still empty. 

org.eclipse.e4.ui.git and some others seem to work well.

Is the issue supposed to be solved for all components in e4?
Comment 58 Denis Roy CLA 2010-07-29 09:52:02 EDT
The import process for e4.resources always fails with:

cvs rlog: Logging e4/org.eclipse.e4.resources/bundles/org.eclipse.ui.ide/icons/full/elcl16
Terminated with fatal signal 11

I'll strace further to hopefully determine why.
Comment 59 Paul Webster CLA 2010-07-29 10:57:54 EDT
(In reply to comment #58)
> The import process for e4.resources always fails with:
> 
> cvs rlog: Logging
> e4/org.eclipse.e4.resources/bundles/org.eclipse.ui.ide/icons/full/elcl16
> Terminated with fatal signal 11
> 
> I'll strace further to hopefully determine why.

When doing a CVS tagging operation on e4/org.eclipse.e4.resources recently, I ran into problems in that directory.  It was complaining about waiting for an anonymous lock so it could proceed.

I don't know if that's the problem, or if the git import is leaving the read-lock there.

PW
Comment 60 Boris Bokowski CLA 2010-07-29 12:12:29 EDT
(In reply to comment #59)
> > The import process for e4.resources always fails with:
> > 
> > cvs rlog: Logging
> > e4/org.eclipse.e4.resources/bundles/org.eclipse.ui.ide/icons/full/elcl16
> > Terminated with fatal signal 11

Looks like it's the same problem as the one we had before, see comment 38 and earlier.
Comment 61 Boris Bokowski CLA 2010-07-29 12:15:38 EDT
Denis, did you apply Bernhard's patch? See comments 49 and 50.
Comment 62 Denis Roy CLA 2010-07-30 14:13:43 EDT
> When doing a CVS tagging operation on e4/org.eclipse.e4.resources recently, I
> ran into problems in that directory.  It was complaining about waiting for an
> anonymous lock so it could proceed.

In the 'Just so you know' dept., the anonymous lock forcibly comes from build.eclipse.org, since it is the only machine which can access pserver using the 'live' CVS files.  Anonymous pserver uses a shadow copy.

 
> I don't know if that's the problem, or if the git import is leaving the
> read-lock there.

Like anonymous pserver, the git import also uses the shadow copy of CVS, so it cannot interfere with committer writes.




> Denis, did you apply Bernhard's patch? See comments 49 and 50.

I have, but it's not really a 'patch' -- it is complete replacement files for 443 files in CVS.  I realize most of these files are Attic files, I'm not comfortable replacing CVS with files in such a brute-force fashion without diffing them first.

Another alternative is to just share the command line/script you used to remove the offending lines (I'm guessing it's an awk/sed one-liner) ?
Comment 63 Denis Roy CLA 2010-07-30 14:15:17 EDT
> it is complete replacement files for 443 files in CVS.

Sorry, I misread... There are 38 files in there with the ~.  I'll just diff them and replace as necessary.  Sorry for the noise.
Comment 64 Denis Roy CLA 2010-07-30 14:57:33 EDT
Created attachment 175606 [details]
Diff log

I've applied the 'patch'.  Attached is a diff log.
Comment 65 John Arthorne CLA 2011-06-22 22:05:37 EDT
This will soon no longer be an issue after e4 migrates fully to git.
Comment 66 Denis Roy CLA 2011-08-12 14:00:45 EDT
> This will soon no longer be an issue after e4 migrates fully to git.

This is no longer an issue since e4 has migrated to git.