Hi Bruno,

In the process of having a build of Hugin on MacOSX 10.8 that is compatible 
with MacOSX 10.6, I found that libpano13 was using a symbol (___progname) 
that was not present on OSX 10.6. 

I only found it in sys_compat_unix.c; I replaced the call from ___progname 
to getprogname() (which is defined in stdlib.h). 
This enables the library to be used on 10.6 and 10.8 without problem.

Though, I do not know if my change produces the same output, it used (from 
what I found) only in file.c:panoPSDResourcesBlockWrite() and I do not know 
how to test it.

Is there any way to test it ? Is this change acceptable and/or can it lead 
to problems ?

I attach the diff to this post.

Matthieu

On Monday, May 13, 2013 11:01:37 PM UTC+2, Bruno Postle wrote:
>
> Hi all, Sourceforge have 'upgraded' the panotools project on 
> sourceforge: http://sourceforge.net/projects/panotools/ 
>
> Mercurial repositories have moved, you can now find the libpano13 
> code here: 
>
>    hg clone http://hg.code.sf.net/p/panotools/libpano13 
>
> So if you have an existing checkout you can either rebase it by 
> editing the .hg/hgrc file, or just fetch a fresh one.  The old 
> Mercurial repository will continue to exist, but is now read-only. 
>
> I took the opportunity to migrate the other panotools projects from 
> Subversion to Mercurial (hooray!), these are: 
>
> Panotools::Script - perl module for manipulating Hugin .pto projects 
>
> GIMP plugin - A GIMP panotools plugin, this needs to maintenance 
>
> ptfilter - Photoshop filters for panotools 
>
> MPRemap - Java moving panoramas remapping 
>
> If there is anything I'm missing, please let me know.  e.g. I 
> haven't migrated the old libpano12 binary-compatible library - Does 
> anyone want this? 
>
> -- 
> Bruno 
>

-- 
-- 
You received this message because you are subscribed to the Google Groups 
"Hugin and other free panoramic software" group.
A list of frequently asked questions is available at: 
http://wiki.panotools.org/Hugin_FAQ
To post to this group, send email to hugin-ptx@googlegroups.com
To unsubscribe from this group, send email to 
hugin-ptx+unsubscr...@googlegroups.com
For more options, visit this group at http://groups.google.com/group/hugin-ptx

--- 
You received this message because you are subscribed to the Google Groups 
"hugin and other free panoramic software" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to hugin-ptx+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.


diff -r 04c52cae988a sys_compat_unix.c
--- a/sys_compat_unix.c	Fri Apr 12 18:56:04 2013 +0200
+++ b/sys_compat_unix.c	Tue May 14 01:37:33 2013 +0200
@@ -20,6 +20,10 @@
 #include "sys_compat.h"
 #include <assert.h>
 
+#ifdef __APPLE__
+#include <stdlib.h>
+#endif
+
 int panoTimeToStrWithTimeZone(char *sTime, int len, struct tm  *time) 
 {
     assert(len >= 11);
@@ -29,6 +33,10 @@
 
 char *panoBasenameOfExecutable(void)
 {
+#ifdef __APPLE__
+    return getprogname();
+#else
     extern char *__progname;
     return __progname;
+#endif
 }

Reply via email to