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)

main.log (70.9 KB) - added by soehn@… 13 years ago.
Portfile-physfs.diff (288 bytes) - added by luiji@… 13 years ago.
Portfile patch
patch-CMakeLists.txt.diff (446 bytes) - added by luiji@… 13 years ago.
Source patch required by Portfile

Download all attachments as: .zip

Change History (11)

Changed 13 years ago by soehn@…

Attachment: main.log added

comment:1 Changed 13 years ago by mf2k (Frank Schima)

Owner: changed from macports-tickets@… to luiji@…
Summary: physfs+universal does not installphysfs +universal does not install

Error is:

:info:build /Developer/usr/bin/llvm-gcc-4.2  -Dphysfs_EXPORTS -DPHYSFS_HAVE_SYS_UCRED_H=1 -D_REENTRANT -D_THREAD_SAFE -DHAVE_ASSERT_H=1 -DPHYSFS_SUPPORTS_ZIP=1 -DPHYSFS_SUPPORTS_7Z=1 -DPHYSFS_SUPPORTS_GRP=1 -DPHYSFS_SUPPORTS_WAD=1 -DPHYSFS_SUPPORTS_HOG=1 -DPHYSFS_SUPPORTS_MVL=1 -DPHYSFS_SUPPORTS_QPAK=1 -DPHYSFS_HAVE_READLINE=1 -O2 -arch x86_64 -arch i386  -O3 -DNDEBUG -arch x86_64 -arch i386 -isysroot / -mmacosx-version-min=10.7 -fPIC -I/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_devel_physfs/physfs/work/physfs-2.0.2/. -I/opt/local/include    -fno-common -g -pipe -Werror -fsigned-char -Wall -fvisibility=hidden -o CMakeFiles/physfs.dir/lzma/C/Archive/7z/7zDecode.o   -c /opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_devel_physfs/physfs/work/physfs-2.0.2/lzma/C/Archive/7z/7zDecode.c
:info:build lipo: can't open input file: /var/tmp//ccyc4stN.out (No such file or directory)

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 installphysfs 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: Portfile-physfs.diff added

Portfile patch

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: newclosed
Note: See TracTickets for help on using tickets.