Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#4550 closed enhancement (fixed)

Remove Perl Module Glib

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

Description (last modified by Fernando de Oliveira)

Only needed as optional by xfconf.

After some discussion, remove, instead of update.

Change History (12)

comment:1 by Fernando de Oliveira, 8 years ago

Apparently only needed by Xfconf-4.10.0 as optional. How to know that it is working? Is it not a candidate for archive?

comment:2 by Pierre Labastie, 8 years ago

I saw that there was a new version, but did not check whether it was needed or not. Actually, there are Perl Bindings for xfconf, but they are built only if three Perl modules are found. From configure --enable-perl-bindings:

*** Required perl modules not found to build perl bindings.  Requirements are:
***
***     ExtUtils::Depends    0.300
***     ExtUtils::PkgConfig  1.000
***     Glib                 1.020
***
*** Please install the required dependencies and run 'configure' again.

So if the Glib module is mentionned, the two others should be too. OTOH, only Glib is in the book.

Consequently: +1 for archiving, and maybe changing the xfconf deps.

comment:3 by Fernando de Oliveira, 8 years ago

Thanks, Pierre.

Actually, it is in the book, under Glib-1.280, the dependency you referred to:

    Glib-1.280 (MD5 sum: 1d81a8aec5f7f1182a96cfaaf119d866)

        ExtUtils::PkgConfig

            ExtUtils::Depends

But only for Glib, it seems. This is what I understand and I may be wrong.

If you and I are correct that only xconf needs it, and no one objects, I agree with what you wrote:

Consequently: +1 for archiving, and maybe changing the xfconf deps.

Perhaps, after waiting a couple of days, if you do not mind, may I ask you to do it? (You know I am dumb about dependencies.)

I am missing your edits and we have a lot to be done. :-)

in reply to:  3 ; comment:4 by Pierre Labastie, 8 years ago

Replying to fo:

Thanks, Pierre.

Actually, it is in the book, under Glib-1.280, the dependency you referred to:

    Glib-1.280 (MD5 sum: 1d81a8aec5f7f1182a96cfaaf119d866)

        ExtUtils::PkgConfig

            ExtUtils::Depends

Should have looked at it.

Perhaps, after waiting a couple of days, if you do not mind, may I ask you to do it? (You know I am dumb about dependencies.)

So am I. And actually, I think it is delicate: Having the above in Perl Modules allows to mention only one dependency in xfconf. Otherwise, you need to put 3 deps, and to specify that one is dependent of the others... Furthermore, I do not know what the xfconf Perl bindings are useful for.

What if we completely suppress that dep?

I am missing your edits and we have a lot to be done. :-)

Thanks! I feel bad to be unable to do more, but it is very busy here (and Bruce likes to give me some jhalfs homework for the week-end ;-)

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

Replying to pierre.labastie:

What if we completely suppress that dep?

That is what I am thinking too: modify the link at xconf to cpan:

http://www.cpan.org/authors/id/T/TS/TSCH/Glib-1.280.tar.gz

and remove all three from the book:

Glib-1.280 (MD5 sum: 1d81a8aec5f7f1182a96cfaaf119d866)

        ExtUtils::PkgConfig

            ExtUtils::Depends

As you, I have no experience with xconf, nor how/when to add/delete perl modules.

Let us wait a couple of days for more comments, if no one objection or volunteer, I suppose (hopefully, pardonnez moi pour insister) you could find some time in your schedule? No hurry, you could just accept, then, and do when time is available.

comment:6 by Fernando de Oliveira, 8 years ago

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

I see you do not want to do it.

I will try, then, tomorrow.

comment:7 by Fernando de Oliveira, 8 years ago

Description: modified (diff)
Summary: Perl Module Glib-1.304Remove Perl Module Glib

comment:8 by Fernando de Oliveira, 8 years ago

Description: modified (diff)

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

Replying to fo:

I see you do not want to do it.

:-)

comment:10 by Fernando de Oliveira, 8 years ago

Description: modified (diff)

comment:11 by Fernando de Oliveira, 8 years ago

Resolution: fixed
Status: assignedclosed

Have commented out everything. If there is a mistake or afterwards we find out it is need, will be easy to revert.

Fixed at r12604.

comment:12 by bdubbs@…, 8 years ago

Milestone: current

Milestone current deleted

Note: See TracTickets for help on using tickets.