unofficial mirror of guile-devel@gnu.org 
 help / color / mirror / Atom feed
* rfc: [build] Overhaul <errno.h>, <signal.h> cpp symbol extraction/checking.
@ 2010-06-17 15:02 Thien-Thi Nguyen
  2010-06-18  8:43 ` Andy Wingo
  0 siblings, 1 reply; 4+ messages in thread
From: Thien-Thi Nguyen @ 2010-06-17 15:02 UTC (permalink / raw)
  To: guile-devel

aka ee2a435c4c5b4dbbbac2006855afecaa23d9585a
aka ttn/misc-maint~2 (at the moment)

The priciple behind this change is: less is more.

OK to push into master?

thi





^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: rfc: [build] Overhaul <errno.h>, <signal.h> cpp symbol extraction/checking.
  2010-06-17 15:02 rfc: [build] Overhaul <errno.h>, <signal.h> cpp symbol extraction/checking Thien-Thi Nguyen
@ 2010-06-18  8:43 ` Andy Wingo
  2010-06-25 12:07   ` Thien-Thi Nguyen
  0 siblings, 1 reply; 4+ messages in thread
From: Andy Wingo @ 2010-06-18  8:43 UTC (permalink / raw)
  To: Thien-Thi Nguyen; +Cc: guile-devel

On Thu 17 Jun 2010 17:02, Thien-Thi Nguyen <ttn@gnuvola.org> writes:

> aka ee2a435c4c5b4dbbbac2006855afecaa23d9585a
> aka ttn/misc-maint~2 (at the moment)
>
> The priciple behind this change is: less is more.
>
> OK to push into master?

Would you mind sending patches using git-format-patch? It's easier to
reply inline that way :-)

I don't really understand this patch, or the current makefile foo thb,
and would appreciate an explanation.

Andy
-- 
http://wingolog.org/



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: rfc: [build] Overhaul <errno.h>, <signal.h> cpp symbol extraction/checking.
  2010-06-18  8:43 ` Andy Wingo
@ 2010-06-25 12:07   ` Thien-Thi Nguyen
  2010-06-28 13:09     ` Andy Wingo
  0 siblings, 1 reply; 4+ messages in thread
From: Thien-Thi Nguyen @ 2010-06-25 12:07 UTC (permalink / raw)
  To: guile-devel

[-- Attachment #1: Type: text/plain, Size: 1087 bytes --]

() Andy Wingo <wingo@pobox.com>
() Fri, 18 Jun 2010 10:43:07 +0200

   Would you mind sending patches using git-format-patch? It's easier to
   reply inline that way :-)

Ah, ok.  I was under the impression that the preferred way was to post
a link (to a commit) so that whoever wants to pull can do so, thus
avoiding big email messages.  Isn't that the "git way"?

   I don't really understand this patch, or the current makefile foo thb,
   and would appreciate an explanation.

I've just pushed ‘ttn/janitor’ (and deleted ‘ttn/misc-maint’), including
the patch below, reformulated (slightly) to add a bit of explanation to
libguile/Makefile.am.  Does that help?  I omit explaining the old way
there, but here's a summary: "essentially (the basic approach) like the
new way , but with more state (files, temporary and permanent) and with
unnecessary modularity".

The branch ‘ttn/janitor’ also has some other patches.  I will post them
individually for review in other threads.

thi


____________________________________________________________

[-- Attachment #2: 0001-build-Overhaul-errno.h-signal.h-cpp-symbol-extractio.patch --]
[-- Type: text/x-diff, Size: 13633 bytes --]

From aa9afa384168c0509b30be0d5f7c61f24f02c729 Mon Sep 17 00:00:00 2001
From: Thien-Thi Nguyen <ttn@gnuvola.org>
Date: Tue, 15 Jun 2010 22:34:27 +0200
Subject: [PATCH 1/8] [build] Overhaul <errno.h>, <signal.h> cpp symbol extraction/checking.

* libguile/Makefile.am (BUILT_SOURCES): Delete cpp_err_symbols.c,
  cpp_sig_symbols.c; add cpp-E.c, cpp-SIG.c.
  (EXTRA_DIST): Delete cpp_signal.c, cpp_errno.c, cpp_err_symbols.in,
  cpp_err_symbols.c, cpp_sig_symbols.c, cpp_sig_symbols.in, cpp_cnvt.awk;
  add cpp-E.syms, cpp-E.c, cpp-SIG.syms, cpp-SIG.c.
  (error.x, posix.x): Update prereq list.
  (cpp_err_symbols.c, cpp_sig_symbols.c): Delete targets.
  (check_signals, check_errnos): Likewise.
  (.syms.c): New pattern rule.
  (chknew-E, chknew-SIG): New targets.
  (MOSTLYCLEANFILES): Delete cpp_err_symbols_here, cpp_err_symbols_diff,
  cpp_err_symbols_new, cpp_sig_symbols_here, cpp_sig_symbols_diff,
  cpp_sig_symbols_new.
* libguile/cpp-E.syms: Rename from libguile/cpp_err_symbols.in.
* libguile/cpp-SIG.syms: Rename from libguile/cpp_sig_symbols.in.
* libguile/error.c (scm_init_error): #include "libguile/cpp-E.c".
* libguile/posix.c (scm_init_posix): #include "libguile/cpp-SIG.c".
* libguile/cpp_cnvt.awk: Delete file.
* libguile/cpp_errno.c: Delete file.
* libguile/cpp_signal.c: Delete file.
---
 libguile/Makefile.am        |  102 +++++++++++++++++++-----------------
 libguile/cpp-E.syms         |  122 +++++++++++++++++++++++++++++++++++++++++++
 libguile/cpp-SIG.syms       |   36 +++++++++++++
 libguile/cpp_cnvt.awk       |    7 ---
 libguile/cpp_err_symbols.in |  122 -------------------------------------------
 libguile/cpp_errno.c        |    9 ---
 libguile/cpp_sig_symbols.in |   36 -------------
 libguile/cpp_signal.c       |    9 ---
 libguile/error.c            |    2 +-
 libguile/posix.c            |    2 +-
 10 files changed, 214 insertions(+), 233 deletions(-)
 create mode 100644 libguile/cpp-E.syms
 create mode 100644 libguile/cpp-SIG.syms
 delete mode 100644 libguile/cpp_cnvt.awk
 delete mode 100644 libguile/cpp_err_symbols.in
 delete mode 100644 libguile/cpp_errno.c
 delete mode 100644 libguile/cpp_sig_symbols.in
 delete mode 100644 libguile/cpp_signal.c

diff --git a/libguile/Makefile.am b/libguile/Makefile.am
index 5eea8dc..6a33473 100644
--- a/libguile/Makefile.am
+++ b/libguile/Makefile.am
@@ -415,7 +415,7 @@ DOT_I_FILES = vm-i-system.i vm-i-scheme.i vm-i-loader.i
 .c.i:
 	$(AM_V_GEN)$(GREP) '^VM_DEFINE' $< > $@
 
-BUILT_SOURCES = cpp_err_symbols.c cpp_sig_symbols.c libpath.h \
+BUILT_SOURCES = cpp-E.c cpp-SIG.c libpath.h \
     version.h scmconfig.h \
     $(DOT_I_FILES) $(DOT_X_FILES) $(EXTRA_DOT_X_FILES)
 
@@ -596,9 +596,8 @@ bin_SCRIPTS = guile-snarf
 noinst_SCRIPTS = guile-doc-snarf guile-snarf-docs guile-func-name-check
 
 EXTRA_DIST = ChangeLog-scm ChangeLog-threads		\
-    ChangeLog-1996-1999 ChangeLog-2000 ChangeLog-2008 cpp_signal.c	\
-    cpp_errno.c cpp_err_symbols.in cpp_err_symbols.c			\
-    cpp_sig_symbols.c cpp_sig_symbols.in cpp_cnvt.awk			\
+    ChangeLog-1996-1999 ChangeLog-2000 ChangeLog-2008	\
+    cpp-E.syms cpp-E.c cpp-SIG.syms cpp-SIG.c			\
     c-tokenize.lex version.h.in						\
     scmconfig.h.top libgettext.h unidata_to_charset.pl libguile.map
 #    $(DOT_DOC_FILES) $(EXTRA_DOT_DOC_FILES) \
@@ -674,8 +673,8 @@ $(DOT_X_FILES) $(EXTRA_DOT_X_FILES): scmconfig.h snarf.h guile-snarf.in
 
 $(DOT_DOC_FILES) $(EXTRA_DOT_DOC_FILES): scmconfig.h snarf.h guile-snarf-docs.in guile_filter_doc_snarfage$(EXEEXT)
 
-error.x: cpp_err_symbols.c
-posix.x: cpp_sig_symbols.c
+error.x: cpp-E.c
+posix.x: cpp-SIG.c
 load.x: libpath.h
 
 alldotdocfiles    = $(DOT_DOC_FILES) $(EXTRA_DOT_DOC_FILES)
@@ -710,50 +709,57 @@ schemelib_DATA = guile-procedures.txt
 ## Add -MG to make the .x magic work with auto-dep code.
 MKDEP = gcc -M -MG $(DEFS) $(AM_CPPFLAGS) $(CPPFLAGS) $(CFLAGS)
 
-cpp_err_symbols.c: cpp_err_symbols.in cpp_cnvt.awk
-	$(AWK) -f $(srcdir)/cpp_cnvt.awk < $(srcdir)/cpp_err_symbols.in > \
-		cpp_err_symbols.tmp
-	mv cpp_err_symbols.tmp cpp_err_symbols.c
-
-cpp_sig_symbols.c: cpp_sig_symbols.in cpp_cnvt.awk
-	$(AWK) -f $(srcdir)/cpp_cnvt.awk < $(srcdir)/cpp_sig_symbols.in > \
-		cpp_sig_symbols.tmp
-	mv cpp_sig_symbols.tmp cpp_sig_symbols.c
-
-## Create a new version of the cpp_sig_symbols.in file, including all SIGXXX
-## macros defined on this platform.
-check_signals:
-	gcc -undef -dM -E $(srcdir)/cpp_signal.c | $(EGREP) ' SIG[A-Z]+' \
-	  | cut -f2 -d' ' | sort > cpp_sig_symbols_here
-	diff -u $(srcdir)/cpp_sig_symbols.in cpp_sig_symbols_here \
-          | $(EGREP) '^\+S' \
-	  | cut -c2- > cpp_sig_symbols_diff
-	if test -s cpp_sig_symbols_diff ; then \
-	  cat $(srcdir)/cpp_sig_symbols.in cpp_sig_symbols_diff \
-	    | sort > cpp_sig_symbols_new ;\
-	  echo "cpp_sig_symbols_new has the following additions:" ;\
-	  cat cpp_sig_symbols_diff ;\
-	else echo "No new symbols found."; \
-	fi
-
-## Likewise for cpp_err_symbols.in.
-check_errnos:
-	gcc -undef -dM -E $(srcdir)/cpp_errno.c | $(EGREP) ' E.+' \
-	  | cut -f2 -d' ' | sort > cpp_err_symbols_here
-	diff -u $(srcdir)/cpp_err_symbols.in cpp_err_symbols_here \
-          | $(EGREP) '^\+E' \
-	  | cut -c2- > cpp_err_symbols_diff
-	if test -s cpp_err_symbols_diff ; then \
-	  cat $(srcdir)/cpp_err_symbols.in cpp_err_symbols_diff \
-	    | sort > cpp_err_symbols_new ;\
-	  echo "cpp_err_symbols_new has the following additions:" ;\
-	  cat cpp_err_symbols_diff ;\
-	else echo "No new symbols found."; \
-	fi
+\f
+# This page is for maintenance of the lists of CPP symbols that are eventually
+# included in error.c (‘errno’ values: E*) and posix.c (signal names: SIG*),
+# in the funcs scm_init_{error,posix}, respectively.
+#
+# The lists (files cpp-{E,SIG}.syms) are not included verbatim, but processed
+# (via pattern rule ‘.syms.c’ below) so that each CPP symbol is transformed to
+# fragment of #ifdef'd C code that defines a Scheme variable of the same name.
+# The resulting files, cpp-{E,SIG}.c, are the ones #include:d.
+#
+# To maintain the (source) lists of symbols, we use a GCC-specific feature
+# to list all symbols #define:d when expanding <{errno,signal}.h>, grep
+# the output for {E,SIG}*, massage the result into a usable form, and
+# construct a union of the current and new symbols.  This is written to
+# files cpp-{E,SIG}.syms.NEW for review.  If things look ok, you can then
+# "mv foo.NEW foo" and commit.  Generating the foo.NEW files and showing
+# an informative message is the job of targets chknew-{E,SIG}.
+
+# For each line in foo.syms, transform:
+#	SYMBOL
+# to:
+#	#ifdef SYMBOL
+#	scm_c_define ("SYMBOL", scm_from_int (SYMBOL));
+#	#endif
+# writing output to foo.c.
+.syms.c:
+	sed 's/.*/#ifdef &\nscm_c_define ("&", scm_from_int (&));\n#endif/' \
+	  $< > $@T
+	mv $@T $@
+
+# Write $(srcdir)/cpp-{E,SIG}.syms.NEW if there are any not-yet-seen
+# ("new" to us) E* or SIG* symbols in <errno.h> or <signal.h>, respectively.
+chknew-E chknew-SIG:
+	@bit=`echo $@ | sed s/^chknew-//` ;			\
+	old="$(srcdir)/cpp-$$bit.syms" ;			\
+	echo "#include <$${bit}.h>"				\
+	 | sed 's/E/errno/;s/SIG/signal/'			\
+	 | gcc -undef -dM -E -					\
+	 | sed 's/^#define //;/^'$$bit'[A-Z][A-Z]*/!d;s/ .*//'	\
+	 | sort | diff -u $$old - | sed '1,2d;/^+/!d;s/^.//'	\
+	 > TMP ;						\
+	if [ -s TMP ] ; then new="$$old.NEW" ;			\
+	  cat $$old TMP | sort > $$new ;			\
+	  echo "$$new: `sed -n '$$=' TMP` new symbol(s)." ;	\
+	  sed 's/^/  /' TMP ;					\
+	else echo "No new symbols found." ;			\
+	fi ; rm TMP
+
+\f
 
 MOSTLYCLEANFILES = \
-	cpp_err_symbols_here cpp_err_symbols_diff cpp_err_symbols_new \
-	cpp_sig_symbols_here cpp_sig_symbols_diff cpp_sig_symbols_new \
 	version.h version.h.tmp \
 	scmconfig.h scmconfig.h.tmp
 
diff --git a/libguile/cpp-E.syms b/libguile/cpp-E.syms
new file mode 100644
index 0000000..67b897c
--- /dev/null
+++ b/libguile/cpp-E.syms
@@ -0,0 +1,122 @@
+E2BIG
+EACCES
+EADDRINUSE
+EADDRNOTAVAIL
+EADV
+EAFNOSUPPORT
+EAGAIN
+EALREADY
+EBADE
+EBADF
+EBADFD
+EBADMSG
+EBADR
+EBADRQC
+EBADSLT
+EBFONT
+EBUSY
+ECHILD
+ECHRNG
+ECOMM
+ECONNABORTED
+ECONNREFUSED
+ECONNRESET
+EDEADLK
+EDEADLOCK
+EDESTADDRREQ
+EDOM
+EDOTDOT
+EDQUOT
+EEXIST
+EFAULT
+EFBIG
+EHOSTDOWN
+EHOSTUNREACH
+EIDRM
+EILSEQ
+EINPROGRESS
+EINTR
+EINVAL
+EIO
+EISCONN
+EISDIR
+EISNAM
+EL2HLT
+EL2NSYNC
+EL3HLT
+EL3RST
+ELIBACC
+ELIBBAD
+ELIBEXEC
+ELIBMAX
+ELIBSCN
+ELNRNG
+ELOOP
+EMFILE
+EMLINK
+EMSGSIZE
+EMULTIHOP
+ENAMETOOLONG
+ENAVAIL
+ENETDOWN
+ENETRESET
+ENETUNREACH
+ENFILE
+ENOANO
+ENOBUFS
+ENOCSI
+ENODATA
+ENODEV
+ENOENT
+ENOEXEC
+ENOLCK
+ENOLINK
+ENOMEM
+ENOMSG
+ENONET
+ENOPKG
+ENOPROTOOPT
+ENOSPC
+ENOSR
+ENOSTR
+ENOSYS
+ENOTBLK
+ENOTCONN
+ENOTDIR
+ENOTEMPTY
+ENOTNAM
+ENOTSOCK
+ENOTTY
+ENOTUNIQ
+ENXIO
+EOPNOTSUPP
+EOVERFLOW
+EPERM
+EPFNOSUPPORT
+EPIPE
+EPROTO
+EPROTONOSUPPORT
+EPROTOTYPE
+ERANGE
+EREMCHG
+EREMOTE
+EREMOTEIO
+ERESTART
+EROFS
+ESHUTDOWN
+ESOCKTNOSUPPORT
+ESPIPE
+ESRCH
+ESRMNT
+ESTALE
+ESTRPIPE
+ETIME
+ETIMEDOUT
+ETOOMANYREFS
+ETXTBSY
+EUCLEAN
+EUNATCH
+EUSERS
+EWOULDBLOCK
+EXDEV
+EXFULL
diff --git a/libguile/cpp-SIG.syms b/libguile/cpp-SIG.syms
new file mode 100644
index 0000000..2118d87
--- /dev/null
+++ b/libguile/cpp-SIG.syms
@@ -0,0 +1,36 @@
+SIGABRT
+SIGALRM
+SIGBREAK
+SIGBUS
+SIGCHLD
+SIGCLD
+SIGCONT
+SIGFPE
+SIGHUP
+SIGILL
+SIGINT
+SIGIO
+SIGIOT
+SIGKILL
+SIGPIPE
+SIGPOLL
+SIGPROF
+SIGPWR
+SIGQUIT
+SIGSEGV
+SIGSTKFLT
+SIGSTOP
+SIGSYS
+SIGTERM
+SIGTRAP
+SIGTSTP
+SIGTTIN
+SIGTTOU
+SIGUNUSED
+SIGURG
+SIGUSR1
+SIGUSR2
+SIGVTALRM
+SIGWINCH
+SIGXCPU
+SIGXFSZ
diff --git a/libguile/cpp_cnvt.awk b/libguile/cpp_cnvt.awk
deleted file mode 100644
index 1e6c094..0000000
--- a/libguile/cpp_cnvt.awk
+++ /dev/null
@@ -1,7 +0,0 @@
-# Converts a list of symbols into C expressions which define the symbols
-# in Guile.
-{
-print "#ifdef " $0;
-print "scm_c_define (\""$0"\", scm_from_int ("$0"));";
-print "#endif"
-}
diff --git a/libguile/cpp_err_symbols.in b/libguile/cpp_err_symbols.in
deleted file mode 100644
index 67b897c..0000000
--- a/libguile/cpp_err_symbols.in
+++ /dev/null
@@ -1,122 +0,0 @@
-E2BIG
-EACCES
-EADDRINUSE
-EADDRNOTAVAIL
-EADV
-EAFNOSUPPORT
-EAGAIN
-EALREADY
-EBADE
-EBADF
-EBADFD
-EBADMSG
-EBADR
-EBADRQC
-EBADSLT
-EBFONT
-EBUSY
-ECHILD
-ECHRNG
-ECOMM
-ECONNABORTED
-ECONNREFUSED
-ECONNRESET
-EDEADLK
-EDEADLOCK
-EDESTADDRREQ
-EDOM
-EDOTDOT
-EDQUOT
-EEXIST
-EFAULT
-EFBIG
-EHOSTDOWN
-EHOSTUNREACH
-EIDRM
-EILSEQ
-EINPROGRESS
-EINTR
-EINVAL
-EIO
-EISCONN
-EISDIR
-EISNAM
-EL2HLT
-EL2NSYNC
-EL3HLT
-EL3RST
-ELIBACC
-ELIBBAD
-ELIBEXEC
-ELIBMAX
-ELIBSCN
-ELNRNG
-ELOOP
-EMFILE
-EMLINK
-EMSGSIZE
-EMULTIHOP
-ENAMETOOLONG
-ENAVAIL
-ENETDOWN
-ENETRESET
-ENETUNREACH
-ENFILE
-ENOANO
-ENOBUFS
-ENOCSI
-ENODATA
-ENODEV
-ENOENT
-ENOEXEC
-ENOLCK
-ENOLINK
-ENOMEM
-ENOMSG
-ENONET
-ENOPKG
-ENOPROTOOPT
-ENOSPC
-ENOSR
-ENOSTR
-ENOSYS
-ENOTBLK
-ENOTCONN
-ENOTDIR
-ENOTEMPTY
-ENOTNAM
-ENOTSOCK
-ENOTTY
-ENOTUNIQ
-ENXIO
-EOPNOTSUPP
-EOVERFLOW
-EPERM
-EPFNOSUPPORT
-EPIPE
-EPROTO
-EPROTONOSUPPORT
-EPROTOTYPE
-ERANGE
-EREMCHG
-EREMOTE
-EREMOTEIO
-ERESTART
-EROFS
-ESHUTDOWN
-ESOCKTNOSUPPORT
-ESPIPE
-ESRCH
-ESRMNT
-ESTALE
-ESTRPIPE
-ETIME
-ETIMEDOUT
-ETOOMANYREFS
-ETXTBSY
-EUCLEAN
-EUNATCH
-EUSERS
-EWOULDBLOCK
-EXDEV
-EXFULL
diff --git a/libguile/cpp_errno.c b/libguile/cpp_errno.c
deleted file mode 100644
index 32556a3..0000000
--- a/libguile/cpp_errno.c
+++ /dev/null
@@ -1,9 +0,0 @@
-/* this file is processed by gcc with special options to extract
-   a list of errno codes.  */
-#include <errno.h>
-
-/*
-  Local Variables:
-  c-file-style: "gnu"
-  End:
-*/
diff --git a/libguile/cpp_sig_symbols.in b/libguile/cpp_sig_symbols.in
deleted file mode 100644
index 2118d87..0000000
--- a/libguile/cpp_sig_symbols.in
+++ /dev/null
@@ -1,36 +0,0 @@
-SIGABRT
-SIGALRM
-SIGBREAK
-SIGBUS
-SIGCHLD
-SIGCLD
-SIGCONT
-SIGFPE
-SIGHUP
-SIGILL
-SIGINT
-SIGIO
-SIGIOT
-SIGKILL
-SIGPIPE
-SIGPOLL
-SIGPROF
-SIGPWR
-SIGQUIT
-SIGSEGV
-SIGSTKFLT
-SIGSTOP
-SIGSYS
-SIGTERM
-SIGTRAP
-SIGTSTP
-SIGTTIN
-SIGTTOU
-SIGUNUSED
-SIGURG
-SIGUSR1
-SIGUSR2
-SIGVTALRM
-SIGWINCH
-SIGXCPU
-SIGXFSZ
diff --git a/libguile/cpp_signal.c b/libguile/cpp_signal.c
deleted file mode 100644
index e740ba5..0000000
--- a/libguile/cpp_signal.c
+++ /dev/null
@@ -1,9 +0,0 @@
-/* this file is processed by gcc with special options to extract
-   a list of signal numbers.  */
-#include <signal.h>
-
-/*
-  Local Variables:
-  c-file-style: "gnu"
-  End:
-*/
diff --git a/libguile/error.c b/libguile/error.c
index bcbcd9c..a582c86 100644
--- a/libguile/error.c
+++ b/libguile/error.c
@@ -285,7 +285,7 @@ scm_misc_error (const char *subr, const char *message, SCM args)
 void
 scm_init_error ()
 {
-#include "libguile/cpp_err_symbols.c"
+#include "libguile/cpp-E.c"
 #include "libguile/error.x"
 }
 
diff --git a/libguile/posix.c b/libguile/posix.c
index 6276478..a28cfa3 100644
--- a/libguile/posix.c
+++ b/libguile/posix.c
@@ -2195,7 +2195,7 @@ scm_init_posix ()
   scm_c_define ("LOCK_NB", scm_from_int (LOCK_NB));
 #endif
 
-#include "libguile/cpp_sig_symbols.c"
+#include "libguile/cpp-SIG.c"
 #include "libguile/posix.x"
 }
 
-- 
1.6.3.2


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: rfc: [build] Overhaul <errno.h>, <signal.h> cpp symbol extraction/checking.
  2010-06-25 12:07   ` Thien-Thi Nguyen
@ 2010-06-28 13:09     ` Andy Wingo
  0 siblings, 0 replies; 4+ messages in thread
From: Andy Wingo @ 2010-06-28 13:09 UTC (permalink / raw)
  To: Thien-Thi Nguyen; +Cc: guile-devel

Hi Thien-Thi,

On Fri 25 Jun 2010 14:07, Thien-Thi Nguyen <ttn@gnuvola.org> writes:

> () Andy Wingo <wingo@pobox.com>
> () Fri, 18 Jun 2010 10:43:07 +0200
>
>    Would you mind sending patches using git-format-patch? It's easier to
>    reply inline that way :-)
>
> Ah, ok.  I was under the impression that the preferred way was to post
> a link (to a commit) so that whoever wants to pull can do so, thus
> avoiding big email messages.  Isn't that the "git way"?

I guess there are many git ways :) Linux uses both, I think. For review
purposes though, inline mails work best for me. I'm happy to change if
there is a better way.

>    I don't really understand this patch, or the current makefile foo thb,
>    and would appreciate an explanation.
>
> I've just pushed ‘ttn/janitor’ (and deleted ‘ttn/misc-maint’), including
> the patch below, reformulated (slightly) to add a bit of explanation to
> libguile/Makefile.am.  Does that help?  I omit explaining the old way
> there, but here's a summary: "essentially (the basic approach) like the
> new way , but with more state (files, temporary and permanent) and with
> unnecessary modularity".

Ah, thank you for this explanation, and that inline to the patch. Feel
free to push this patch to master.

Cheers,

Andy
-- 
http://wingolog.org/



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2010-06-28 13:09 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-06-17 15:02 rfc: [build] Overhaul <errno.h>, <signal.h> cpp symbol extraction/checking Thien-Thi Nguyen
2010-06-18  8:43 ` Andy Wingo
2010-06-25 12:07   ` Thien-Thi Nguyen
2010-06-28 13:09     ` Andy Wingo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).