Re: [gamin] make Gamin support Solaris





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




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]