Bug 606746 - gettext-0.18 does not require CVS
Summary: gettext-0.18 does not require CVS
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: gettext
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Jens Petersen
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-06-22 11:54 UTC by Daniel Walsh
Modified: 2019-03-22 07:02 UTC (History)
7 users (show)

Fixed In Version: gettext-0.18.1.1-1.1.fc13
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-07-06 17:13:47 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Daniel Walsh 2010-06-22 11:54:36 UTC
rpm -q --whatrequires cvs
cvsweb-3.0.6-9.fc12.noarch
gettext-0.18.1.1-1.fc14.x86_64
fedora-packager-0.4.2.1-1.fc14.noarch


I added cgroup support to policycoreutils and this ended up pulling in the cvs package.

Prompting this comment.

             mooninite - 2010-06-22 03:32:42 (karma 0)
             This was an unexpected surprise. yum update results in
             a pull in of libcgroup, which pulls in redhat-lsb,
             which pulls in gettext and pax, which pulls in cvs.
             Why do desktop systems need cvs now? Maybe the thought
             of including libcgroup (or fix its deps) should be
             rethought.

Comment 1 Peter Robinson 2010-06-22 12:47:09 UTC
Agreed. This is ridiculous.... but unfortunately its a long running issue! Started here in bug # 469555 which removed the dep. But then it caused issues with autopoint which needs it, details in bug # 517361

Comment 2 Daniel Walsh 2010-06-22 12:55:56 UTC
Daniel, isn't there a better way then requiring cvs for gettext?

Comment 3 Daniel Berrangé 2010-06-22 13:32:03 UTC
libcgroup does not have any direct dependency on gettext.  It does however have a dependency on redhat-lsb, and redhat-lsb has a dependency on gettext.

The 'gettext' RPM is a package that contains primarily development tools and should only be required by things at build time. The actual gettext APIs are all in the standard GlibC RPM.  

As such, IMHO, the problem that needs focusing on is the redhat-lsb <-> gettext dep.  Either redhat-lsb needs changing to not require the gettext RPM, or we need to split off the command line tools that relevant for production environments,  from the tools that are only needed in development environments and have separate RPM for them.

Separately from that, libcgroup should just be changed to remove any need for a dep on redhat-lsb by using the native Fedora init functions. redhat-lsb is just one of those crazy packages there for standards compliance, for 3rd party vendors to ship cross-distro packages. No app in the Fedora repo has any compelling need to use this package & indeed on my system libcgroup is the only thing that uses it.

Comment 4 Mamoru TASAKA 2010-06-22 13:36:44 UTC
Well,
With previous gettext-0.17-17.fc14.i686 /usr/bin/autopoint says:
--------------------------------------------------------------
   342  # Check availability of the CVS program.
   343  (cvs -v) >/dev/null 2>/dev/null || func_fatal_error "cvs program not found"
--------------------------------------------------------------

Now gettext-0.18.1.1/NEWS says:
--------------------------------------------------------------
    10  * Installation options:
    11    The configure option --with-cvs is deprecated. The 'autopoint' program will
    12    now use the 'git' program by default to compress its archive. If the
    13    configure option --without-git is specified, 'autopoint' will not rely on
    14    'git', but will instead rely on a locally installed a 3 MB large archive.
--------------------------------------------------------------
and actually gettext-0.18.1.1-1.fc14.i686 /usr/bin/autopoint says:
--------------------------------------------------------------
   389  case "git" in
   390    dir)
   399      ;;
   400  
   401    cvs)
   448      (cvs -v) >/dev/null 2>/dev/null || func_fatal_error "cvs program not found"
   512      ;;
   513  
   514    git)
   515      # Check availability of the git program.
   516      (git --version) >/dev/null 2>/dev/null || func_fatal_error "git program not found"
---------------------------------------------------------------
so now not cvs but git is mandatory....

I think requiring all people using gettext to install git should be
avoided. IMO srpms which actually use autopoint should additional
"BR: git", or in more radical way, autopoint should be split out
into subpackages.

Comment 5 Peter Robinson 2010-06-22 13:41:48 UTC
>     13    configure option --without-git is specified, 'autopoint' will not
> rely on
>     14    'git', but will instead rely on a locally installed a 3 MB large
> archive.

Surely the above option means you can configure without git as well.

> ---------------------------------------------------------------
> so now not cvs but git is mandatory....
> 
> I think requiring all people using gettext to install git should be
> avoided. IMO srpms which actually use autopoint should additional
> "BR: git", or in more radical way, autopoint should be split out
> into subpackages.    

Agreed.

Comment 6 Mamoru TASAKA 2010-06-22 13:53:22 UTC
(In reply to comment #5)
> >     13    configure option --without-git is specified, 'autopoint' will not
> > rely on
> >     14    'git', but will instead rely on a locally installed a 3 MB large
> > archive.
> 
> Surely the above option means you can configure without git as well.

To be clearly, this means to avoid the dependency for git,
we have to recompile "gettext" itself with adding "--without-git".

Comment 7 Jens Petersen 2010-06-23 05:01:27 UTC
I am planning to subpackage autopoint for f14.

Comment 8 Jens Petersen 2010-06-23 05:04:11 UTC
Should gettext-autopoint (debian just calls the package autopoint)
not use git either?

Comment 9 Mamoru TASAKA 2010-06-23 05:24:20 UTC
(In reply to comment #8)
> Should gettext-autopoint (debian just calls the package autopoint)
> not use git either?    

Currently I have no opinion for this, because I don't know
in detail what autopoint actually is. I just think that
"gettext" should not depend on cvs (or git) (and suggested
autopoint splitting).

Comment 10 Jens Petersen 2010-06-23 06:16:15 UTC

*** This bug has been marked as a duplicate of bug 574031 ***

Comment 11 Jens Petersen 2010-06-23 06:49:22 UTC
Actually let me use this bug for the incorrect cvs requires.

Comment 12 Jens Petersen 2010-06-23 07:38:31 UTC
Should be fixed in gettext-0.18.1.1-2.fc14:

http://koji.fedoraproject.org/koji/buildinfo?buildID=179311

Comment 13 Fedora Update System 2010-06-23 07:49:48 UTC
gettext-0.18.1.1-1.1.fc13,blender-2.49b-9.fc13,gambas-1.0.19-13.fc13 has been submitted as an update for Fedora 13.
http://admin.fedoraproject.org/updates/gettext-0.18.1.1-1.1.fc13,blender-2.49b-9.fc13,gambas-1.0.19-13.fc13

Comment 14 Michael Cronenworth 2010-06-23 13:25:09 UTC
(In reply to comment #3)
> Separately from that, libcgroup should just be changed to remove any need for a
> dep on redhat-lsb by using the native Fedora init functions. redhat-lsb is just
> one of those crazy packages there for standards compliance, for 3rd party
> vendors to ship cross-distro packages. No app in the Fedora repo has any
> compelling need to use this package & indeed on my system libcgroup is the only
> thing that uses it.    

Bug 607189 opened.

Comment 15 Fedora Update System 2010-06-23 17:40:51 UTC
blender-2.49b-9.fc13, gettext-0.18.1.1-1.1.fc13, gambas-1.0.19-13.fc13 has been pushed to the Fedora 13 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update blender gettext gambas'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/gettext-0.18.1.1-1.1.fc13,blender-2.49b-9.fc13,gambas-1.0.19-13.fc13

Comment 16 Fedora Update System 2010-07-06 17:13:32 UTC
blender-2.49b-9.fc13, gettext-0.18.1.1-1.1.fc13, gambas-1.0.19-13.fc13 has been pushed to the Fedora 13 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.