Bug 1230479 - BuildRequires on gcc not glibc-headers, and Requires on glibc-headers should be removed.
Summary: BuildRequires on gcc not glibc-headers, and Requires on glibc-headers should ...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: pcre2
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Petr Pisar
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1230505
TreeView+ depends on / blocked
 
Reported: 2015-06-11 03:14 UTC by Carlos O'Donell
Modified: 2015-06-22 11:25 UTC (History)
1 user (show)

Fixed In Version: pcre2-10.20-0.1.RC1.fc23
Clone Of:
Environment:
Last Closed: 2015-06-22 11:25:24 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Carlos O'Donell 2015-06-11 03:14:13 UTC
Two issues:

Please replace the BuildRequires: glibc-headers with BuildRequires: gcc. The new Fedora packaging guidelines require you define all the things you need in the buildroot to build [1].

Given that this is a C library it should BuildRequires: gcc, and that is sufficient to bring in all of the required core C libraries and headers to compile C code.

The reason I'm asking is because glibc-headers may or may not be removed in the future and it isn't sufficient to guarantee the package builds.

[1] https://fedoraproject.org/wiki/Packaging:Guidelines#BuildRequires_2

*and*

The devel package should not Requires on glibc-headers.

The development libraries and headers will be used by developers via BuildRequires, but along with BuildRequires: gcc or BuildRequires: gcc-gfortran or BuildRequires: gcc-c++ to be able to compile their code.

The compiler will ensure the standard libraries and headers are installed to make sure the library headers are complete e.g. that #include <stdio.h> works.

There is no need to Require: glibc-headers. The reason I'm asking for the removal of glibc-headers is that the package may or may not go away in the future depending on sub-package restructuring, and it is not needed.


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