Ticket #3142 (closed bug: fixed)

Opened 16 months ago

Last modified 8 months ago

cvs program malfunction

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

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

haiku-cvs-bug.png Download (7.0 KB) - added by bhaible 16 months ago.
screenshot of commands and error message
cvs-1.12.13-build.log Download (68.3 KB) - added by scottmc 13 months 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?

Change History

Changed 16 months ago by bhaible

screenshot of commands and error message

  Changed 16 months ago by scottmc

  • cc haiku@… added

  Changed 16 months ago by kaliber

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

  Changed 16 months ago by anevilyak

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

follow-up: ↓ 7   Changed 14 months ago by scottmc

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

  Changed 13 months ago by scottmc

@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

  Changed 13 months ago by bhaible

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.

Changed 13 months ago by scottmc

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   Changed 13 months ago by scottmc

Replying to scottmc:

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

I just rebuilt with r29182 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.

  Changed 8 months ago by scottmc

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.

  Changed 8 months ago by scottmc

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

  Changed 8 months ago by stippi

  • status changed from new to closed
  • resolution set to fixed

Ok, thanks for the update! Fixed in r31443.

Note: See TracTickets for help on using tickets.