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 >> [email protected] >> 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 _______________________________________________ Gamin-list mailing list [email protected] http://mail.gnome.org/mailman/listinfo/gamin-list
