Menu

#12 php-clamav.spec should not use setup -c

v1.0 (example)
closed-fixed
ARGOS
None
5
2015-08-07
2015-08-04
No

The spec file uses the "-c" option to setup, which creates a directory. Then it unarchives itself into that directory, which also creates a directory with the name of the archive. The problem with that is that phpize cannot find config.m4 then. I made the following change, which helped.

[bbuilder@block01 ~]$ diff rpmbuild/SPECS/php-clamav.spec Z/php-clamav-0.15.8/SPECS/php-clamav.spec 
30c30
< %setup -q php-%{php_libname}-%{version}
---
> %setup -q -c php-%{php_libname}-%{version}

Discussion

  • Tommy McNeely

    Tommy McNeely - 2015-08-04

    Additionally, it appears to require openssl(-devel)...

    [bbuilder@block01 SPECS]$ diff php-clamav.spec php-clamav-fix.spec 
    18a19
    > BuildRequires: openssl-devel
    20a22
    > Requires:      openssl
    22a25
    > Conflicts:     php-clamav
    30c33
    < %setup -q -c php-%{php_libname}-%{version}
    ---
    > %setup -q php-%{php_libname}-%{version}
    
     
  • ARGOS

    ARGOS - 2015-08-07

    Thanks for your report, I have commit your change to SVN.

     
  • ARGOS

    ARGOS - 2015-08-07
    • status: open --> closed-fixed
    • assigned_to: ARGOS
     

Log in to post a comment.

Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.