On Tue, Jul 11, 2006 at 09:01:37PM -0400, James R. Van Zandt wrote:
> 
> >   Please install the static library and header files as part of the
> >   package. It appears to be possible by changing the commented out
> >   dh_install in debian/rules to
> >
> >        dh_install SRC/libPCx.a usr/lib
> >        dh_install */*.h usr/include/pcx
> 
> I've implemented this and uploaded the new package.  Hope this helps!
> 
> >   BTW, I would like to build a free GPL licensed package that uses
> >   pcx but I'm reluctant to do so because of the condition that the
> >   distributor may not charge for the media. If you are in touch with
> >   upstream, would you mention that there are those who would be
> >   grateful if they reconsidered this restriction?
> 
> Not sure what you're asking for here.  I would think you could develop
> scripts that call PCx without having to distribute PCx itself - just
> let Debian do that part.  Anyway, I am not often in touch with the
> developers.  FWIW I'm attaching the last message I got from them.  
> 
> The PCx web page is now: 
> http://www-fp.mcs.anl.gov/otc/Tools/PCx/
> 
>            - Jim Van Zandt

Thanks for implementing my suggested changes to the pcx package. 

The worrisome thing about the PCx license terms is that they claim to
apply to derivative works. Also, I have hopes of my package being
included in Debian some day, which wouldn't be possible if it depended
on a non-free package.

My package doesn't run PCx in a script but calls the C library
functions directly. My best idea lately is to have it dlopen the
library if found but otherwise run without it, which I hope wouldn't
defy either Debian policy or copyright law.

The pcx package could be distributed separately from mine by Debian as
you recommend, but I'll need a dynamic library (because the static
library libPCx.a can't be dlopened). This patch to debian/rules makes
it build one, with the PCx executable binary remaining statically
linked. (The PCx build script appears to ignore the CFLAGS environment
variable but observes CC and FC.)

*** ../../pcx-1.1.17/debian/rules       Thu Jul 20 16:09:54 2006
--- rules       Thu Jul 20 16:21:04 2006
***************
*** 34,40 ****
        dh_testdir
  
        # Add here commands to compile the package.
!       CFLAGS="$(CFLAGS)" ./build
  
        touch build-stamp
  
--- 34,41 ----
        dh_testdir
  
        # Add here commands to compile the package.
!       CFLAGS="$(CFLAGS)" CC="gcc -fPIC" FC="f77 -fPIC" ./build
!       gcc -shared -Wl,-soname,libPCx.so -o libPCx.so.1.1.17 */*.o -lc
  
        touch build-stamp
  
***************
*** 70,78 ****
        dh_installchangelogs debian/changelog-upstream.html
        dh_installdocs README README.mex DOC/PCx-user.ps.gz 
        dh_installexamples mps/25fv47.mps mps/afiro.mps debian/README.examples
!       dh_install SRC/libPCx.a usr/lib
        dh_install */*.h usr/include/pcx
- #     dh_install
  #     dh_installmenu
  #     dh_installdebconf       
  #     dh_installlogrotate
--- 71,78 ----
        dh_installchangelogs debian/changelog-upstream.html
        dh_installdocs README README.mex DOC/PCx-user.ps.gz 
        dh_installexamples mps/25fv47.mps mps/afiro.mps debian/README.examples
!       dh_install lib* SRC/libPCx.a usr/lib
        dh_install */*.h usr/include/pcx
  #     dh_installmenu
  #     dh_installdebconf       
  #     dh_installlogrotate
***************
*** 84,90 ****
  #     dh_installinfo
        dh_installman debian/PCx.1 debian/mps-pcx.5 
        dh_link
!       dh_strip
        dh_compress
        dh_fixperms
  #     dh_perl
--- 84,90 ----
  #     dh_installinfo
        dh_installman debian/PCx.1 debian/mps-pcx.5 
        dh_link
!       dh_strip --exclude libPCx.so
        dh_compress
        dh_fixperms
  #     dh_perl

I also had to make some changes to postinst and prerm to create and
remove the symbolic link correctly, even though I thought that's what
dh_makeshlibs and/or ldconfig were for.

*** ../../pcx-1.1.17/debian/postinst    Thu Jul 20 16:09:54 2006
--- postinst    Thu Jul 20 16:13:18 2006
***************
*** 24,29 ****
--- 24,30 ----
  
  case "$1" in
      configure)
+         ldconfig
        update-alternatives --install                   \
            /usr/share/man/man5/mps.5.gz mps.5.gz       \
            /usr/share/man/man5/mps-pcx.5.gz 40 
*** ../../pcx-1.1.17/debian/prerm       Thu Jul 20 16:09:54 2006
--- prerm       Thu Jul 20 16:13:18 2006
***************
*** 17,22 ****
--- 17,24 ----
  
  case "$1" in
      remove|upgrade|deconfigure)
+         ldconfig
+       rm /usr/lib/libPCx.so
        update-alternatives --remove mps.5.gz \
            /usr/share/man/man5/mps-pcx.5.gz
  
  
I would be very appreciative if you made these further changes to the
package.

Best regards,

Dennis


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to