Fixing some build issue on OSX

Hi Paulo

I have commit a fix that cause some bug at build time. You can found the commit to merge here:
https://github.com/mathk/smalltalk/commit/1fa5efe01f

The issue was that the environ global was remove from libc on osx and replace by the _NSGetEnviron.
I have only be able to test on 10.7. So I have no idea if that is compatible with 10.6

Also the MessgaeLookup.st file was missing in the kernel package.xml

Thanks in advance

--
Mathieu

[bug] excessive RPATH to std. libdir

Issue status update for 
http://smalltalk.gnu.org/node/635
Post a follow up: 
http://smalltalk.gnu.org/project/comments/add/635

 Project:      GNU Smalltalk
 Version:      <none>
 Component:    Build
 Category:     bug reports
 Priority:     normal
 Assigned to:  Unassigned
 Reported by:  ildar.mulyukov
 Updated by:   ildar.mulyukov
 Status:       active

Hi! I just built GST. As the result:

objdump -p .libs/gst-tool | grep RPATH
 RPATH                /usr/lib

It's usually not so bad, but lately is treated as a flaw.

[bug] Object: nil error: did not understand #methodsFor:

Issue status update for 
http://smalltalk.gnu.org/node/636
Post a follow up: 
http://smalltalk.gnu.org/project/comments/add/636

 Project:      GNU Smalltalk
 Version:      <none>
 Component:    VisualGST
 Category:     bug reports
 Priority:     normal
 Assigned to:  Unassigned
 Reported by:  ildar.mulyukov
 Updated by:   ildar.mulyukov
 Status:       active

GIT HEAD.

$ gst-browser 
Recompiling classes...
Recompiling class: GTK.GtkRequisition class
Recompiling selector: #sizeof
Recompiling classes...
Recompiling classes...
Recompiling classes...
Recompiling class: GTK.GdkColor class
Recompiling selector: #sizeof
Recompiling class: GTK.GdkGCValues class
Recompiling selector: #sizeof
Recompiling class: GTK.GtkTextAppearance class
Recompiling selector: #sizeof

Re: [bug] excessive RPATH to std. libdir

Issue status update for 
http://smalltalk.gnu.org/project/issue/635
Post a follow up: 
http://smalltalk.gnu.org/project/comments/add/635

 Project:      GNU Smalltalk
 Version:      <none>
 Component:    Build
 Category:     bug reports
 Priority:     normal
 Assigned to:  Unassigned
 Reported by:  ildar.mulyukov
 Updated by:   bonzinip
-Status:       active
+Status:       invalid

Patching out rpath is something that distros should do, not upstream. 
See the Fedora packaging guidelines for more information.

--disable-rpath only takes care of libiconv, not of libtool, so you
need the second technique described there.

Re: [bug] Better implementation of mutex in MethodDictionary

Issue status update for 
http://smalltalk.gnu.org/project/issue/634
Post a follow up: 
http://smalltalk.gnu.org/project/comments/add/634

 Project:      GNU Smalltalk
 Version:      <none>
 Component:    VM
 Category:     bug reports
 Priority:     normal
 Assigned to:  Unassigned
 Reported by:  MrGwen
 Updated by:   MrGwen
 Status:       active
 Attachment:   http://smalltalk.gnu.org/files/issues/mutex.patch (479 bytes)

With the patch it's better

[bug] Better implementation of mutex in MethodDictionary

Issue status update for 
http://smalltalk.gnu.org/node/634
Post a follow up: 
http://smalltalk.gnu.org/project/comments/add/634

 Project:      GNU Smalltalk
 Version:      <none>
 Component:    VM
 Category:     bug reports
 Priority:     normal
 Assigned to:  Unassigned
 Reported by:  MrGwen
 Updated by:   MrGwen
 Status:       active

current implementation of MethodDictionary>>#mutex is not atomic,
so I propose a better implementation like Process>>#interruptLock
implementation

Re: [task] More Delay tests

Issue status update for 
http://smalltalk.gnu.org/project/issue/633
Post a follow up: 
http://smalltalk.gnu.org/project/comments/add/633

 Project:      GNU Smalltalk
 Version:      <none>
 Component:    Testsuite
 Category:     tasks
 Priority:     normal
 Assigned to:  Unassigned
 Reported by:  MrGwen
 Updated by:   MrGwen
 Status:       active
 Attachment:   http://smalltalk.gnu.org/files/issues/delayTests.patch (2.97 KB)

hmmm I can't upload the patch

[task] More Delay tests

Issue status update for 
http://smalltalk.gnu.org/node/633
Post a follow up: 
http://smalltalk.gnu.org/project/comments/add/633

 Project:      GNU Smalltalk
 Version:      <none>
 Component:    Testsuite
 Category:     tasks
 Priority:     normal
 Assigned to:  Unassigned
 Reported by:  MrGwen
 Updated by:   MrGwen
 Status:       active

Hi Paolo,

I've made more small tests for Delay, feel free to add them (or not).
If you have any remarks they are welcome.

Cheers,
Gwen

Re: [bug] Error when building gnutls-wrapper

Issue status update for 
http://smalltalk.gnu.org/project/issue/571
Post a follow up: 
http://smalltalk.gnu.org/project/comments/add/571

 Project:      GNU Smalltalk
 Version:      <none>
 Component:    Build
 Category:     bug reports
 Priority:     normal
 Assigned to:  Unassigned
 Reported by:  Carlo_Bramini
 Updated by:   bonzinip
-Status:       active
+Status:       fixed

I have not yet made a release, but I did fix the placement of the
#defines in commit f38d4fa9ef4ac49a725e617c60ec26d685210e69.

I'll change the LDADD now, thanks!

Re: [bug] Error when building gnutls-wrapper

Issue status update for 
http://smalltalk.gnu.org/project/issue/571
Post a follow up: 
http://smalltalk.gnu.org/project/comments/add/571

 Project:      GNU Smalltalk
 Version:      <none>
 Component:    Build
 Category:     bug reports
 Priority:     normal
 Assigned to:  Unassigned
 Reported by:  Carlo_Bramini
 Updated by:   Carlo_Bramini
-Status:       fixed
+Status:       active
 Attachment:   http://smalltalk.gnu.org/files/issues/sm.txt (1.3 KB)

It looks like the bug still exists in newer version and the patch seems
wrong.
The SHUT_RDWR was still not found, probably because you added the
various #define in the block where __MSVCRT__ is undefined.
I wouldn't use hardcoded values, I would simply define instead:

    * define SHUT_RD   SD_RECEIVE
    * define SHUT_WR   SD_SEND
    * define SHUT_RDWR SD_BOTH

In my opinion there is no need to do #ifndef tests on them: if that
code block is relative to Windows, then it is sure that these macro do
not exist.

[PATCH] Make gst-package --test new_package.xml work

Hi Paolo,

$ gst-package --test package.xml

fails if the package is just being installed. This is because 
PackageDirectories>>#refreshStarList: is never called. My patch is to flush 
the PackageLoader state, and then refresh, I am not sure if I break something 
in regard to svn/git/http packages.

holger

_______________________________________________
help-smalltalk mailing list
help-smalltalk <at> gnu.org
https://lists.gnu.org/mailman/listinfo/help-smalltalk