Opened 16 years ago

Closed 15 years ago

#3142 closed bug (fixed)

cvs program malfunction

Reported by: bhaible Owned by: axeld
Priority: normal Milestone: R1
Component: - General Version: R1/pre-alpha1
Keywords: Cc: haiku@…
Blocked By: Blocking:
Platform: All

Description

The following totally standard use of the 'cvs' program yields an error. It does not do so on Linux or MacOS X.

rm -rf w repo mkdir w repo cd w mkdir d touch d/a b c cvs -Q -d cd ../repo && pwd init cvs -Q -d cd ../repo && pwd import -m imp m M M0

Attachments (2)

haiku-cvs-bug.png (7.0 KB ) - added by bhaible 16 years ago.
screenshot of commands and error message
cvs-1.12.13-build.log (68.3 KB ) - added by scottmc 16 years ago.
Here's a log of my building of cvs-1.12.13, maybe someone can spot one of the failed checks that might be related to this not working right?

Download all attachments as: .zip

Change History (12)

by bhaible, 16 years ago

Attachment: haiku-cvs-bug.png added

screenshot of commands and error message

comment:1 by scottmc, 16 years ago

Cc: haiku@… added

comment:2 by kaliber, 16 years ago

CVS doesn't work because of broken mbrtowc() in libroot. It can be workarounded by disabling mbrtowc in cvs:

  1. ./configure --prefix=/boot/common
  2. remove line: HAVE_MBRTOWC in config.h
  3. make && make install

comment:3 by anevilyak, 16 years ago

Could you please try again with hrev28768 or higher? That revision updated Haiku's CVS package with kaliber's workaround.

comment:4 by scottmc, 16 years ago

TODO: rebuild cvs-1.12.13 once mbrtowc() is fixed, perhaps this was fixed with hrev28970

comment:5 by scottmc, 16 years ago

@Bruno, I'm guessing your command should be broken up like this?

rm -rf w repo 
mkdir w repo 
cd w mkdir d 
touch d/a b c 
cvs -Q -d cd ../repo && pwd init 
cvs -Q -d cd ../repo && pwd import -m imp m M M0

comment:6 by bhaible, 16 years ago

The commands were the following. (Unfortunately the wiki formatting messed it up.)

rm -rf w repo
mkdir w repo
cd w
mkdir d
touch d/a b c
cvs -Q -d `cd ../repo && pwd` init
cvs -Q -d `cd ../repo && pwd` import -m imp m M M0

See also in the attached image.

by scottmc, 16 years ago

Attachment: cvs-1.12.13-build.log added

Here's a log of my building of cvs-1.12.13, maybe someone can spot one of the failed checks that might be related to this not working right?

in reply to:  4 comment:7 by scottmc, 16 years ago

Replying to scottmc:

TODO: rebuild cvs-1.12.13 once mbrtowc() is fixed, perhaps this was fixed with hrev28970

I just rebuilt with hrev29182 and the mbrtowc() seems to work now, but didn't fix this issue. I've posted the terminal output from my build in case someone spots an issue there that might cause the failure listed here. Note the output of this test is at the end of my build log.

comment:8 by scottmc, 15 years ago

I no longer see this error with a new build of cvs with the wchar fix in place. Leave open until the wchar fix is moved into trunk.

comment:9 by scottmc, 15 years ago

This ticket can be closed now as the wchar fix is in trunk and the cvs optional package has been updated.

comment:10 by stippi, 15 years ago

Resolution: fixed
Status: newclosed

Ok, thanks for the update! Fixed in hrev31443.

Note: See TracTickets for help on using tickets.