Bug 1313455 - [calligra-l10n] Please add Supplements tag in specfile for langpacks
Summary: [calligra-l10n] Please add Supplements tag in specfile for langpacks
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: calligra-l10n
Version: 24
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Rex Dieter
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1314406
TreeView+ depends on / blocked
 
Reported: 2016-03-01 15:57 UTC by Parag Nemade
Modified: 2016-03-30 08:44 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2016-03-30 08:44:45 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Add Supplements for langpacks guidelines (19.84 KB, patch)
2016-03-01 15:57 UTC, Parag Nemade
no flags Details | Diff
Add Supplements for langpacks guidelines (21.57 KB, patch)
2016-03-12 15:15 UTC, Parag Nemade
no flags Details | Diff

Description Parag Nemade 2016-03-01 15:57:08 UTC
Created attachment 1131987 [details]
Add Supplements for langpacks guidelines

Description of problem:
As per newly approved https://fedoraproject.org/wiki/Packaging:Langpacks in https://fedorahosted.org/fpc/ticket/593#comment:18 , I would like to request to commit the attached patch in F24+ branches.

Comment 1 Parag Nemade 2016-03-12 14:43:30 UTC
Well, I can also apply this patch as I am provenpackager. Should I apply?

Comment 2 Rex Dieter 2016-03-12 14:51:33 UTC
In this case, I'd like to keep calligra-l10n mergeable with other branches, so I think that means adding conditionals around these, something like:

%if 0%{?fedora} > 23
Supplements: ...
%endif

If you can do that, I don't mind you committing it.

Comment 3 Parag Nemade 2016-03-12 15:15:08 UTC
Created attachment 1135588 [details]
Add Supplements for langpacks guidelines

Revised patch that will add conditionals so that backporting of this commit will not add supplements to older branches.

Comment 4 Parag Nemade 2016-03-30 08:44:45 UTC
Built in F24+


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