Opened 13 years ago
Closed 13 years ago
#31158 closed update (fixed)
class-dump 3.3.4 update
Reported by: | 0xced (Cédric Luthi) | Owned by: | macports-tickets@… |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | maintainer | Cc: | 0xced (Cédric Luthi), mklein-de (Michael Klein) |
Port: | class-dump |
Description
I asked Waqar Malik — the current maintainer of the port — if he was OK for me to take ownership of the class-dump port and he said OK. So here is an updated class-dump 3.3.4 port.
Attachments (6)
Change History (16)
Changed 13 years ago by 0xced (Cédric Luthi)
Changed 13 years ago by 0xced (Cédric Luthi)
Attachment: | patch-dyld-info-compat.h.diff added |
---|
comment:1 Changed 13 years ago by mf2k (Frank Schima)
Port: | class-dump added |
---|---|
Version: | 2.0.2 |
comment:2 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)
I looked at a diff between the existing portfile and your new one. You've changed the whitespace of the portfile, in addition to updating to version 3.3.4. These changes should be made separately. Provide a diff that makes only functional changes, then once that's committed, you can provide a second diff that makes only whitespace changes if desired. (Or the other way around, as you wish.)
You've also made the port require Snow Leopard or newer, whereas before it required Leopard or newer. I'm surprised the developers would increase the minimum system requirements in a bugfix release. Are you sure this port won't work on Leopard anymore?
Changed 13 years ago by 0xced (Cédric Luthi)
Attachment: | v3.3.4.diff added |
---|
Changed 13 years ago by 0xced (Cédric Luthi)
Attachment: | spacing.diff added |
---|
comment:3 Changed 13 years ago by 0xced (Cédric Luthi)
Here are two unified diffs, one for v3.3.4 update and one for fixing tabs to spaces, in that order.
And yes, class-dump 3.3.4 requires Snow Leopard.
comment:5 Changed 13 years ago by com-macosforge@…
class-dump 3.3.4 isn't building on Snow Leopard either for me (see attachment).
comment:6 Changed 13 years ago by 0xced (Cédric Luthi)
Can you please upload your loader.h
file located at /Developer/SDKs/MacOSX10.6.sdk/usr/include/mach-o/loader.h
? I have noticed that this file, part of the 10.6 SDK differs between different Xcode versions. I have Xcode 3.2.6, 4.1 and 4.2 and each time loader.h
(all from the 10.6 SDK) is different! Also, please specify the exact version of Xcode installed in /Developer.
Changed 13 years ago by girl@…
comment:7 Changed 13 years ago by girl@…
Hi Cedric, I am having the same build error on 10.6.8 -- am running version XCode Build version 4... if that helps.
comment:10 Changed 13 years ago by mf2k (Frank Schima)
Resolution: | → fixed |
---|---|
Status: | new → closed |
Please supply a unified diff of the Portfile. In the future, please fill in the Port field.