[Bug-gnulib] poll proposed patch for BUILT_SOURCES, $(POLL_H), $@

[Paul Eggert]

I propose the following patch to modules/poll.  It's not needed for
the coreutils merge, since coreutils doesn't use poll, but it's better
to be consistent.

2003-08-17  Paul Eggert  <address@hidden>

        * modules/poll: Add BUILT_SOURCES.  Use $(POLL_H), not @address@hidden
        Prefer "$@" to "poll.h" in rule for building poll.h.
        All this is for consistency with alloca and fnmatch, after
        the latter were merged from gnulib.

Index: modules/poll
===================================================================
RCS file: /cvsroot/gnulib/gnulib/modules/poll,v
retrieving revision 1.2
diff -p -u -r1.2 poll
--- modules/poll        7 Jun 2003 16:41:48 -0000       1.2
+++ modules/poll        17 Aug 2003 03:16:43 -0000
@@ -12,14 +12,16 @@ configure.ac:
 gl_FUNC_POLL
 
 Makefile.am:
+BUILT_SOURCES += $(POLL_H)
 EXTRA_DIST += poll_.h
 
 # We need the following in order to create an <poll.h> when the system
 # doesn't have one.
-all-local $(lib_OBJECTS): @POLL_H@
+BUILT_SOURCES += $(POLL_H)
+all-local $(lib_OBJECTS): $(POLL_H)
 poll.h: poll_.h
-       cp $(srcdir)/poll_.h poll.h-t
-       mv poll.h-t poll.h
+       cp $(srcdir)/poll_.h address@hidden
+       mv address@hidden $@
 MOSTLYCLEANFILES += poll.h poll.h-t
 
 Include: