Opened 13 years ago
Closed 13 years ago
#30586 closed defect (fixed)
physfs does not build on 10.7
Reported by: | soehn@… | Owned by: | luiji@… |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.0.1 |
Keywords: | lion | Cc: | |
Port: | physfs |
Description
I tried to install dosbox. Physfs+universal did not install. I have attached the log file.
Attachments (3)
Change History (11)
Changed 13 years ago by soehn@…
comment:1 Changed 13 years ago by mf2k (Frank Schima)
Owner: | changed from macports-tickets@… to luiji@… |
---|---|
Summary: | physfs+universal does not install → physfs +universal does not install |
comment:2 Changed 13 years ago by jmroot (Joshua Root)
Actually this is the first error (because of -Werror):
:info:build cc1: warnings being treated as errors :info:build /opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_devel_physfs/physfs/work/physfs-2.0.2/platform/macosx.c: In function â__PHYSFS_platformGetThreadIDâ: :info:build /opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_devel_physfs/physfs/work/physfs-2.0.2/platform/macosx.c:358: warning: âMPCurrentTaskIDâ is deprecated (declared at //System/Library/Frameworks/CoreServices.framework/Frameworks/CarbonCore.framework/Headers/Multiprocessing.h:466)
comment:3 Changed 13 years ago by jmroot (Joshua Root)
Keywords: | lion added |
---|---|
Summary: | physfs +universal does not install → physfs does not build on 10.7 |
comment:4 Changed 13 years ago by luiji@…
So, the first error...
:info:build lipo: can't open input file: /var/tmp//ccyc4stN.out (No such file or directory)
...seems to be similar to the problem on this forum. It appears to be that you need to explicitly specify the architectures? I'm not really good with universal builds...
As for the second error relating to MPCurrentTaskID, I'm going to send a message to Ryan Gordan's inbox about that.
comment:5 Changed 13 years ago by jmroot (Joshua Root)
The reason lipo can't open the input file is because the compilation failed earlier because of the warning. It's just a deprecated function, it should still work fine. Upstream will probably want to switch to something more modern in a future release, but all you need to do in the port is remove -Werror.
comment:6 Changed 13 years ago by luiji@…
Okay, cool. So:
- Ryan Gordan has already fixed this in Mercurial; the next release will not need a patch
- I am attaching a Portfile patch and a source patch required by it
Changed 13 years ago by luiji@…
Attachment: | patch-CMakeLists.txt.diff added |
---|
Source patch required by Portfile
comment:7 Changed 13 years ago by luiji@…
Patches uploaded. Now all we need is someone with SVN access and everything will be gravy. Unless I made a terrible mistake...
comment:8 Changed 13 years ago by mf2k (Frank Schima)
Resolution: | → fixed |
---|---|
Status: | new → closed |
Error is: