Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#6076 closed enhancement (fixed)

Test-Command-0.11

Reported by: ken@… Owned by: ken@…
Priority: normal Milestone: 7.7
Component: BOOK Version: SVN
Severity: normal Keywords:
Cc:

Description

feh requires this if its test suite is to run.

I am ambivalent about this (I do not find feh useful, for me ImageMagick's display is more convenient), but I'm willing to add it if nobody objects).

I might have created the thin end of the wedge by recommending the test perl module deps for biber, and therefore adding them, but in that case the latest of biber broke at runtime and its make test showed that.

If no objections, I ill do this with the other perl module tickets I've got _after_ I've completed my fresh builds.

Change History (9)

comment:1 by ken@…, 9 years ago

Owner: changed from blfs-book@… to ken@…
Status: newassigned

comment:2 by bdubbs@…, 9 years ago

Is this a complicated build? If it's only CMMI, is would prefer just leaving it as an external dependency with a note that it is required for testing.

The point is that I would prefer to avoid obscure, one time packages in the book. It just makes maintenance harder and few people will use it.

comment:3 by Fernando de Oliveira, 9 years ago

No, it is perl module, that is the reason I suggested it. But if you still think it is not worth it and Ken agrees, it is OK with me.

comment:4 by bdubbs@…, 9 years ago

Sorry. It wasn't apparent that it was a Perl module. Adding that is OK. To be honest, I always use the alternate cpan instructions because it's still from source and a lot faster.

comment:5 by ken@…, 9 years ago

This one will take me a little longer to test (it's a server package, I use it for SpamAssassin, will be building a VM to test my server stuff). But I took a look to see what in the book uses this, in case that was a quicker way to test, and only Bind optionally uses this, so we already have a precedent for listing optional dependencies which are perl modules.

comment:6 by ken@…, 9 years ago

Resolution: fixed
Status: assignedclosed

Done in r15423.

comment:7 by Fernando de Oliveira, 9 years ago

Thank you very much, ĸen.

in reply to:  5 ; comment:8 by ken@…, 9 years ago

Replying to ken@…:

This one will take me a little longer to test (it's a server package, I use it for SpamAssassin, will be building a VM to test my server stuff). But I took a look to see what in the book uses this, in case that was a quicker way to test, and only Bind optionally uses this, so we already have a precedent for listing optional dependencies which are perl modules.

Sorry, that was intended for a different ticket.

in reply to:  8 comment:9 by Fernando de Oliveira, 9 years ago

Replying to ken@…:

Replying to ken@…:

This one will take me a little longer to test (it's a server package

Sorry, that was intended for a different ticket.

Form me, no worries. I assumed that had happened. :-)

Thanks, again.

Note: See TracTickets for help on using tickets.