Bug 761409 - remove fedora-packager dependency from -tools in RHEL
Summary: remove fedora-packager dependency from -tools in RHEL
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: fontpackages
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Nicolas Mailhot
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-12-08 09:26 UTC by Parag Nemade
Modified: 2012-06-01 17:04 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-06-01 17:04:05 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
this will disable -tools on rhel (547 bytes, patch)
2011-12-08 09:27 UTC, Parag Nemade
no flags Details | Diff
This will remove fedora-packager dependency from -tools subpackage (1.24 KB, patch)
2012-05-21 06:13 UTC, Parag Nemade
no flags Details | Diff
This will remove fedora-packager dependency from -tools subpackage (1.39 KB, patch)
2012-05-21 08:31 UTC, Parag Nemade
no flags Details | Diff

Description Parag Nemade 2011-12-08 09:26:03 UTC
Description of problem:
I think on RHEL we don't need -tools subpackage.That pulls fedora-packager package as a dependency also which is not available on RHEL.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Parag Nemade 2011-12-08 09:27:43 UTC
Created attachment 542448 [details]
this will disable -tools on rhel

Comment 2 Nicolas Mailhot 2011-12-12 09:04:16 UTC
would it be possible to conditionnalize only the problematic dep and not the whole tools package ?

Comment 3 Bill Nottingham 2011-12-12 21:41:28 UTC
Why? We don't use the -tools package in RHEL.

Comment 4 Parag Nemade 2011-12-21 07:04:17 UTC
nim-nim,
   Can patch be applied then?

Comment 5 Parag Nemade 2011-12-27 05:24:09 UTC
nim-nim,
    I am trying to understand for what purpose fedora-packager is needed as a dependency in -tools package. Can you help me to understand it?

Comment 6 Nicolas Mailhot 2012-01-18 19:53:31 UTC
(In reply to comment #5)
> nim-nim,
>     I am trying to understand for what purpose fedora-packager is needed as a
> dependency in -tools package. Can you help me to understand it?

I think it's only used to compute package owners via /usr/bin/fedoradev-pkgowners in /usr/bin/repo-font-audit

So the correct thing for RHEL is probably to patch this part out without breaking the rest of the script, and remove the dep from the tools subpackage

Comment 8 Parag Nemade 2012-01-28 10:12:43 UTC
nim-nim,
    Is it ok if for RHEL, consolidate function be skipped from repo-font-audit?

Comment 9 Akira TAGOH 2012-02-03 07:58:38 UTC
nim-nim,

That would be nice if you can clarify how providing -tools in RHEL is useful. unfortunately even I have no idea for that.

Comment 10 Parag Nemade 2012-05-07 06:04:53 UTC
nim-nim ping

Comment 11 Parag Nemade 2012-05-16 04:01:51 UTC
I tried to remove the call for fedoradev-pkgowners and consolidated output got changed but this resulted in all subsequent test outputs in file to get moved one column left. Its not easy to modify all the script. Better to drop -tools subpackage for RHEL.

Comment 12 Parag Nemade 2012-05-16 09:01:08 UTC
nim-nim,
     I come to conclusion that adding conditional for RHEL to remove -tools is the only good option instead to modify all the parts of the script repo-font-audit. We need to patch spec file.

Comment 13 Nicolas Mailhot 2012-05-16 10:20:30 UTC
Can't you just replace the call with something that inserts an empty value ?

That will produce and empty column, but it's good enough to let the rest of the tests run, no?

Comment 14 Jens Petersen 2012-05-17 00:44:15 UTC
Parag, perhaps you can attach your current patch for review/comments?

Comment 15 Parag Nemade 2012-05-17 03:20:22 UTC
nim-nim,
   if that is what you like to see then I will prepare a patch for that.

Comment 16 Parag Nemade 2012-05-21 06:13:43 UTC
Created attachment 585729 [details]
This will remove fedora-packager dependency from -tools subpackage

Comment 17 Parag Nemade 2012-05-21 08:31:06 UTC
Created attachment 585761 [details]
This will remove fedora-packager dependency from -tools subpackage

Comment 18 Fedora Update System 2012-05-23 09:59:59 UTC
fontpackages-1.44-4.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/fontpackages-1.44-4.fc17

Comment 19 Fedora Update System 2012-05-24 15:35:06 UTC
Package fontpackages-1.44-4.fc17:
* should fix your issue,
* was pushed to the Fedora 17 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing fontpackages-1.44-4.fc17'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2012-8303/fontpackages-1.44-4.fc17
then log in and leave karma (feedback).

Comment 20 Fedora Update System 2012-06-01 17:04:05 UTC
fontpackages-1.44-4.fc17 has been pushed to the Fedora 17 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.