Opened 8 months ago

Last modified 6 months ago

#69517 new defect

npth @1.7 does not build on PPC Tiger, Mac OS X 10.4.11, because patching fails

Reported by: ballapete (Peter "Pete" Dyballa) Owned by:
Priority: Normal Milestone:
Component: ports Version: 2.9.1
Keywords: tiger ppc Cc:
Port: npth

Description

--->  Extracting npth-1.7.tar.bz2
Executing:  cd "/opt/local/var/macports/build/_opt_local_var_macports_sources_nue.de.rsync.macports.org_macports_release_tarballs_ports_devel_npth/npth/work" && /opt/local/bin/lbzip2 -dc '/opt/local/var/macports/distfiles/npth/npth-1.7.tar.bz2' | /usr/bin/gnutar --no-same-owner -xf - 
--->  Applying patches to npth
--->  Applying use-darwin-compat.diff
Executing:  cd "/opt/local/var/macports/build/_opt_local_var_macports_sources_nue.de.rsync.macports.org_macports_release_tarballs_ports_devel_npth/npth/work/npth-1.7" && /usr/bin/patch -p0 < '/opt/local/var/macports/sources/nue.de.rsync.macports.org/macports/release/tarballs/ports/devel/npth/files/use-darwin-compat.diff'
patching file src/Makefile.am
Hunk #1 FAILED at 31.
1 out of 1 hunk FAILED -- saving rejects to file src/Makefile.am.rej
patching file src/npth.c
Command failed:  cd "/opt/local/var/macports/build/_opt_local_var_macports_sources_nue.de.rsync.macports.org_macports_release_tarballs_ports_devel_npth/npth/work/npth-1.7" && /usr/bin/patch -p0 < '/opt/local/var/macports/sources/nue.de.rsync.macports.org/macports/release/tarballs/ports/devel/npth/files/use-darwin-compat.diff'
Exit code: 1
Error: Failed to patch npth: command execution failed
Error: See /opt/local/var/macports/logs/_opt_local_var_macports_sources_nue.de.rsync.macports.org_macports_release_tarballs_ports_devel_npth/npth/main.log for details.

Cause is

diff npth-1.6/src/Makefile.am npth-1.7/src/Makefile.am
33c33
< libnpth_la_SOURCES = npth.h npth.c npth-sigev.c
---
> libnpth_la_SOURCES = npth.c npth-sigev.c

Attachments (3)

main.log (9.6 KB) - added by ballapete (Peter "Pete" Dyballa) 8 months ago.
Main.log from PPC Tiger, Mac OS X 10.4.11
use-darwin-compat-1.7.diff (521 bytes) - added by ballapete (Peter "Pete" Dyballa) 8 months ago.
Patch for npth v. 1.7
npth-patch-for-tiger-leopard.diff (1.9 KB) - added by ballapete (Peter "Pete" Dyballa) 8 months ago.
New patch for unnamed semaphores on Tiger and Leopard

Download all attachments as: .zip

Change History (10)

Changed 8 months ago by ballapete (Peter "Pete" Dyballa)

Attachment: main.log added

Main.log from PPC Tiger, Mac OS X 10.4.11

Changed 8 months ago by ballapete (Peter "Pete" Dyballa)

Attachment: use-darwin-compat-1.7.diff added

Patch for npth v. 1.7

comment:1 Changed 8 months ago by ballapete (Peter "Pete" Dyballa)

With the attached and updated patch npth @1.7 built.

comment:2 Changed 8 months ago by ballapete (Peter "Pete" Dyballa)

This version identifies itself as 1.7-unknown – would it make sense to rename it to 1.7-Darwin? It is away from the original version…

comment:3 Changed 8 months ago by ballapete (Peter "Pete" Dyballa)

I made the npth maintainers aware of the changes needed to make it work on Tiger and Leopard: https://dev.gnupg.org/T7057.

comment:4 Changed 8 months ago by ballapete (Peter "Pete" Dyballa)

NIIBE Yutaka created a git patch that solves the problem on Tiger. I then converted this patch into unified format. it's attached here. gpg-agent launches and can be killed as well. Portfile need to be changed from

platform darwin {
    if {${os.major} < 10 || (${os.major} == 10 && ${configure.build_arch} eq "ppc")} {
        post-extract {
            copy ${filespath}/darwin_compat.h ${worksrcpath}/src/
            copy ${filespath}/darwin_compat.c ${worksrcpath}/src/
        }   
        patchfiles-append   use-darwin-compat.diff
        use_autoreconf      yes 
        autoreconf.args     -fvi
    }   
}

to simple

platform darwin {
    if {${os.major} < 10 || (${os.major} == 10 && ${configure.build_arch} eq "ppc")} {
        patchfiles-append   npth-unnamed-semaphores-on-tiger-and-leopard.diff
        use_autoreconf      yes
        autoreconf.args     -fvi
    }
}

Changed 8 months ago by ballapete (Peter "Pete" Dyballa)

New patch for unnamed semaphores on Tiger and Leopard

comment:5 Changed 6 months ago by ballapete (Peter "Pete" Dyballa)

It is installed since over a month: npth @1.7_1 (active) requested_variants='' platform='darwin 8' archs='ppc' date='2024-03-26T10:23:09+0100'.

comment:6 Changed 6 months ago by ryandesign (Ryan Carsten Schmidt)

But what you've installed is something custom to your machine. You have version 1.7 revision 1 but I don't know what that is; the newest we have in MacPorts is 1.7 revision 0.

comment:7 Changed 6 months ago by ballapete (Peter "Pete" Dyballa)

It's built with the corrected/updated patch.

Note: See TracTickets for help on using tickets.