Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#6636 closed defect (fixed)

CVS-1.11.23: comment out test suite

Reported by: Fernando de Oliveira Owned by: Fernando de Oliveira
Priority: normal Milestone: 7.8
Component: BOOK Version: SVN
Severity: normal Keywords:
Cc:

Description

Test suit fails at basicb-21, less than 1 SBU from start. It is supposed to run for over one hour.

I am using --with-rsh=ssh.

This can be fixed by

https://projects.archlinux.org/svntogit/packages.git/plain/trunk/cvs-1.11.23-sanity.patch?h=packages/cvs

Unfortunately, again, after a loooong time, it hangs for ate least two reasons.

It tries to do an ssh connection to 127.0.0.1.

First, I was in one terminal, following check.log, because during the test no message goes to the console and didn't notice, when the standard ssh trust question appeared in the other terminal. After yes, message was the expected broken pipe.

Second, it started asking my password and repeated about 10 times, and I stopped the test.

One more problem is that the sed expression

's/g=rw,o=r$/g=r,o=r/'

does nothing.

I replaced by

's/g=rw,o=rw/g=r,o=r/'

but have no idea if it is correct. Perhaps that is the reason for the hung up.

Perhaps, the switches discussed for the tests and in Optional dependencies:

--with-editor=<desired text editor>

--with-rsh=ssh or empty (?)

should be mentioned in Command Explanations.

Also the text explaining the tests problems could be improved.

I think this is all.

Please, I would really prefer somebody else to confirm or unconfirm the problems and fix the issues.

Change History (2)

comment:1 by Fernando de Oliveira, 6 years ago

Owner: changed from blfs-book@… to Fernando de Oliveira
Status: newassigned

OK, I will try something.

Last edited 6 years ago by Fernando de Oliveira (previous) (diff)

comment:2 by Fernando de Oliveira, 6 years ago

Resolution: fixed
Status: assignedclosed
Summary: cvs-1.11.23: test suite problems and command explanations missingCVS-1.11.23: comment out test suite

Decided to comment out the test suite and include

    <para>This package does not come with a working test suite.</para>

If somebody wants to fix or finds that I am wrong, please change it.

Fixed at r16161.

Note: See TracTickets for help on using tickets.