Re: [gamin] make Gamin support Solaris

[Lin Ma <Lin Ma Sun COM>]

Doug Scott wrote:
> Lin Ma wrote:
> > Hi there,
> >
> > This is the patch for Solaris File Events Notification (FEN) backend. 
> > Pls review it.
> > Currently all the test scenarios except 4.tst can be passed. But 
> > 4.tst seems to conflict to 9.tst, and I can find the precise 
> > definition about monitoring directory, could you tell me the reason? 
> > I will work another patch accordingly.
> >
> > Thanks,
> > lin
> >
> > ------------------------------------------------------------------------
> >
> > _______________________________________________
> > Gamin-list mailing list
> > Gamin-list gnome org
> > http://mail.gnome.org/mailman/listinfo/gamin-list
>
> Lin,
>   Good to see the FEN port coming along. A quick note -
>
> The LDFLAGS here only needs to be changed when using the Solaris 
> native 'ld'. Thus it needs a 'if SOLARIS' condition around it, 
> otherwise it will break a Linux build. This one is probably my fault 
> originally :( I will test it when I get sxce b74 installed on my machine.
No. I should thanks for your jobs on GAMIN first :-) .
Thanks your reminder, I'm sure there are other problems especially in my 
code, I will update the patch after I get all the feedback.

Thanks,

lin
> Doug
>
> -libgamin_1_la_LDFLAGS = 
> -Wl,--version-script=$(srcdir)/gamin_sym.version \
> +libgamin_1_la_LDFLAGS = -Wl,-M$(srcdir)/gamin_sym.version \
>                         -version-info @GAMIN_VERSION_INFO@ @THREAD_LIBS@
>
> libfam_la_SOURCES = $(libgamin_1_la_SOURCES)
> libfam_la_LIBADD = $(libgamin_1_la_LIBADD)
> -libfam_la_LDFLAGS = 
> -Wl,--version-script=$(srcdir)/gamin_sym.version    \
> +libfam_la_LDFLAGS = -Wl,-M$(srcdir)/gamin_sym.version    \
>                     -version-info @FAM_VERSION_INFO@ @THREAD_LIBS@

--
x82120 / +86 10 82618200