#2 fix the build and minor issus
Closed 5 years ago by sergiomb. Opened 5 years ago by sergiomb.
Unknown source master  into  master

file modified
+15 -3
@@ -1,9 +1,11 @@

  # Enabled by default

  %bcond_without tests

  

+ %global optflags %(echo %{optflags} -fsigned-char)

+ 

  Name:       gdcm

  Version:    2.8.8

- Release:    2%{?dist}

+ Release:    3%{?dist}

  Summary:    Grassroots DiCoM is a C++ library to parse DICOM medical files

  License:    BSD

  URL:        http://gdcm.sourceforge.net/wiki/index.php/Main_Page
@@ -117,9 +119,13 @@

  used this library with python

  

  %prep

- %autosetup -n GDCM-%{version} -S git

+ %autosetup -n GDCM-%{version} -S git -N

  # Data source

  %setup -n GDCM-%{version} -q -T -D -a 1

+ %patch -p1 -P 1 2 3 4 5 7

+ %if 0%{?fedora} > 29

+ %patch6 -p1 -b .poppler.0.73.0

+ %endif

  

  # Fix cmake command

  sed -i.backup 's/add_dependency/add_dependencies/' Utilities/doxygen/CMakeLists.txt
@@ -160,7 +166,6 @@

      -DGDCM_DATA_ROOT=../gdcmData/ \

      -DGDCM_BUILD_EXAMPLES:BOOL=ON \

      -DGDCM_DOCUMENTATION:BOOL=ON \

-     -DGDCM_PDF_DOCUMENTATION:BOOL=OFF \

      -DGDCM_WRAP_PYTHON:BOOL=ON \

      -DPYTHON_EXECUTABLE=%{__python3} \

      -DGDCM_INSTALL_PYTHONMODULE_DIR=%{python3_sitearch} \
@@ -280,6 +285,13 @@

  %{python3_sitearch}/__pycache__/%{name}*

  

  %changelog

+ * Fri Feb 22 2019 Sérgio Basto <sergio@serjux.com> - 2.8.8-3

+ - Manually-specified variables were not used by the project:

+   GDCM_PDF_DOCUMENTATION

+ - Patch for poppler breaks builds for previous releases.

+ - Add -fsigned-char to the build flags. The compiler complaints make sense if

+   char is unsigned.

+ 

  * Mon Feb 18 2019 Ankur Sinha <ankursinha AT fedoraproject DOT org> - 2.8.8-2

  - Enable tests

  

Manually-specified variables were not used by the project: GDCM_PDF_DOCUMENTATION
Patch for poppler breaks builds for previous releases.
Add -fsigned-char to the build flags. The compiler complaints make sense if
char is unsigned.

Pull-Request has been closed by sergiomb

5 years ago