New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #30586 (closed defect: fixed)

Opened 23 months ago

Last modified 23 months ago

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

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

Change History

Changed 23 months ago by soehn@…

comment:1 Changed 23 months ago by macsforever2000@…

  • Owner changed from macports-tickets@… to luiji@…
  • Summary changed from physfs+universal does not install to physfs +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 23 months ago by jmr@…

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 23 months ago by jmr@…

  • Keywords lion added
  • Summary changed from physfs +universal does not install to physfs does not build on 10.7

comment:4 Changed 23 months 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 23 months ago by jmr@…

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 23 months 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 23 months ago by luiji@…

Portfile patch

Changed 23 months ago by luiji@…

Source patch required by Portfile

comment:7 Changed 23 months 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 23 months ago by macsforever2000@…

  • Status changed from new to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.