[repo.or.cz] wmaker-crm.git branch next updated: wmaker-0.95.6-247-g2086eef

2015-04-18 Thread crmafra
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project wmaker-crm.git.

The branch, next has been updated
  discards  45374b598fe2917eb678a8f18418ef940b47f825 (commit)
  discards  6f67f80aa411adc960c4431dc4aff46652ac9228 (commit)
  discards  e6a2edbc9c31404d8f22e1923603a1422067cffd (commit)
  discards  68aee92278bd6414c43059d1e26699fcf005 (commit)
  discards  b11cd0cf1656150c4d9cee58631d1de7723b9e82 (commit)
  discards  ae92bccb14432a8e4ab6619f5171a7b56506e5e6 (commit)
   via  2086eefb7e5c658bf0f99feb2aa4b118b188e2b1 (commit)
   via  842e145e0aeef633b7d0f938f57098469fee7cfe (commit)
   via  b4cf94aacd0f9fcffd802b0a8e47e0d162ca1f20 (commit)
   via  2ebfcd9c5c9350d89f2346304ed8c02883020785 (commit)
   via  1cef020eb322f1bc2978d49ec2d846dd9d1cc15d (commit)
   via  30fe0fb05f6fd0030155c2643a93a562c8278711 (commit)
   via  d1c9f791d870d88ec4f02e31b29600e1afbaa58a (commit)

This update added new revisions after undoing existing revisions.  That is
to say, the old revision is not a strict subset of the new revision.  This
situation occurs when you --force push a change and generate a repository
containing something like this:

 * -- * -- B -- O -- O -- O (45374b598fe2917eb678a8f18418ef940b47f825)
\
 N -- N -- N (2086eefb7e5c658bf0f99feb2aa4b118b188e2b1)

When this happens we assume that you've already had alert emails for all
of the O revisions, and so we here report only the revisions in the N
branch from the common base, B.

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -
http://repo.or.cz/w/wmaker-crm.git/commit/2086eefb7e5c658bf0f99feb2aa4b118b188e2b1

commit 2086eefb7e5c658bf0f99feb2aa4b118b188e2b1
Author: Christophe CURIS 
Date:   Sat Apr 18 23:56:19 2015 +0200

texi2txt: add workaround for a known bug in "mawk"

As reported by Douglas Torrance, the script that generates the plain text
documentation from the texinfo source would crash due to the cross-
references when the version of awk used is "mawk".

This is actually a known bug in mawk:
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=172774

It looks like the bug have been fixed in mawk, but Debian is still
providing a broken version. To avoid problems, this patch is introducing a
workaround in the matching pattern that causes the issue every time it is
being used.

Signed-off-by: Christophe CURIS 

diff --git a/script/generate-txt-from-texi.sh b/script/generate-txt-from-texi.sh
index 5554e6e..3800c72 100755
--- a/script/generate-txt-from-texi.sh
+++ b/script/generate-txt-from-texi.sh
@@ -212,12 +212,12 @@ function end_conditional(name,  local_i) {
 # with @value; they can also be defined from command-line (-D)
 # they are stored in the global array "variable[name]"
 function set_variable(line,  local_idx, local_name, local_value) {
-  gsub(/^[ \t]*/, "", line);
+  gsub(/^[ \t]+/, "", line);
   local_idx = match(line, /[ \t]/);
   if (local_idx > 0) {
 local_name  = substr(line, 1, local_idx - 1);
 local_value = substr(line, local_idx + 1);
-gsub(/^[ \t]*/, "", local_value);
+gsub(/^[ \t]+/, "", local_value);
   } else {
 local_name  = line;
 local_value = "";
@@ -473,7 +473,7 @@ function new_node(args,local_nb, local_arr, 
local_i) {
   if ((local_nb < 1) || (local_nb > 4)) {
 report_error("bad number of argument " local_nb " for @node at line " NR);
   }
-  gsub(/^[ \t]*/, "", local_arr[1]);
+  gsub(/^[ \t]+/, "", local_arr[1]);
   gsub(/[ \t]*$/, "", local_arr[1]);
   if (local_arr[1] == "") {
 report_error("missing node name for @node at line " NR);
@@ -572,7 +572,7 @@ function generate_cross_reference(args, cmd,  
local_nb, local_arr, local
 
   local_arr[1] = execute_commands(local_arr[1]);
   for (local_i = 1; local_i <= local_nb; local_i++) {
-gsub(/^[ \t]*/, "", local_arr[local_i]);
+gsub(/^[ \t]+/, "", local_arr[local_i]);
 gsub(/[ \t]*$/, "", local_arr[local_i]);
   }
 

http://repo.or.cz/w/wmaker-crm.git/commit/842e145e0aeef633b7d0f938f57098469fee7cfe

commit 842e145e0aeef633b7d0f938f57098469fee7cfe
Author: Christophe CURIS 
Date:   Sat Apr 18 20:17:30 2015 +0200

doc: re-generate the top directory documentation when "make dist" is invoked

As pointed by Douglas Torrance, there are some case where the 2 docs
INSTALL-WMAKER and README.i18n are not re-generated properly, one of the
cases being if the user grabs the sources from Git to make a source package
only (ie, without compiling anything).

This patch adds a hook to "make dist" so it will re-generate the docs if
needed. Because the "dist-hook" is run after Automake has copied the files

[PATCH 1/1] merge with ae92bccb14432a8e4ab6619f5171a7b56506e5e6

2015-04-18 Thread Christophe CURIS
(fix typo)
---
 doc/build/Compilation.texi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/doc/build/Compilation.texi b/doc/build/Compilation.texi
index cb19a6d..4db3ae0 100644
--- a/doc/build/Compilation.texi
+++ b/doc/build/Compilation.texi
@@ -224,7 +224,7 @@ Get the @emph{GNU} version from 
@uref{http://www.gnu.org/software/gettext/}
 
 This library can be used by the @emph{WINGs} toolkit to improve support for 
@emph{UTF-8} and for
 languages written in right-to-left direction, in some widgets.
-You have to explicitly ask for its support though (@pxref{Configure Options}).
+You have to explicitly ask for its support through (@pxref{Configure Options}).
 You can get it from @uref{http://www.pango.org/Download}
 
 @item @emph{libbsd}
-- 
2.1.4


-- 
To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.


[PATCH 1/1] texi2txt: add workaround for a known bug in "mawk"

2015-04-18 Thread Christophe CURIS
As reported by Douglas Torrance, the script that generates the plain text
documentation from the texinfo source would crash due to the cross-
references when the version of awk used is "mawk".

This is actually a known bug in mawk:
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=172774

It looks like the bug have been fixed in mawk, but Debian is still
providing a broken version. To avoid problems, this patch is introducing a
workaround in the matching pattern that causes the issue every time it is
being used.

Signed-off-by: Christophe CURIS 
---
 script/generate-txt-from-texi.sh | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/script/generate-txt-from-texi.sh b/script/generate-txt-from-texi.sh
index 5554e6e..3800c72 100755
--- a/script/generate-txt-from-texi.sh
+++ b/script/generate-txt-from-texi.sh
@@ -212,12 +212,12 @@ function end_conditional(name,  local_i) {
 # with @value; they can also be defined from command-line (-D)
 # they are stored in the global array "variable[name]"
 function set_variable(line,  local_idx, local_name, local_value) {
-  gsub(/^[ \t]*/, "", line);
+  gsub(/^[ \t]+/, "", line);
   local_idx = match(line, /[ \t]/);
   if (local_idx > 0) {
 local_name  = substr(line, 1, local_idx - 1);
 local_value = substr(line, local_idx + 1);
-gsub(/^[ \t]*/, "", local_value);
+gsub(/^[ \t]+/, "", local_value);
   } else {
 local_name  = line;
 local_value = "";
@@ -473,7 +473,7 @@ function new_node(args,local_nb, local_arr, 
local_i) {
   if ((local_nb < 1) || (local_nb > 4)) {
 report_error("bad number of argument " local_nb " for @node at line " NR);
   }
-  gsub(/^[ \t]*/, "", local_arr[1]);
+  gsub(/^[ \t]+/, "", local_arr[1]);
   gsub(/[ \t]*$/, "", local_arr[1]);
   if (local_arr[1] == "") {
 report_error("missing node name for @node at line " NR);
@@ -572,7 +572,7 @@ function generate_cross_reference(args, cmd,  
local_nb, local_arr, local
 
   local_arr[1] = execute_commands(local_arr[1]);
   for (local_i = 1; local_i <= local_nb; local_i++) {
-gsub(/^[ \t]*/, "", local_arr[local_i]);
+gsub(/^[ \t]+/, "", local_arr[local_i]);
 gsub(/[ \t]*$/, "", local_arr[local_i]);
   }
 
-- 
2.1.4


-- 
To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.


Re: [PATCH 14/23] texi2txt: add support for making cross-references in the document

2015-04-18 Thread Christophe

- Christophe  a écrit :
> 
> - Douglas Torrance  a écrit :
> > On 04/06/2015 10:58 AM, Christophe CURIS wrote:
> > > The texinfo format provides 3 commands @ref, @xref and @pxref to make 
> > > cross
> > > references to existing @nodes in the document; it also provides a command
> > > @anchor to place arbitrary targets for cross-reference.
> > >
> > > Because these will be handy for the Installation Manual that already does
> > > some references, this patch implements the 4 commands:
> > >
> > > [...]
> > 
> > There must be some GNU extensions in here somewhere, as I get the 
> > following error without gawk when running the script on Compilation.texi:
> > 
> > Error: cross reference to undefined node/anchor "ConfigureOptions" found 
> > at line 227
> 
> 
> Unfortunately I was not able to reproduce so far on #next; I tried with:
>   awk --posix
>   awk --traditional
>   nawk
> 
> None of them is complaining. Looking at the message, it seems there is a 
> space missing between the 2 words, so it looks like a difference in parsing. 
> What's strange is that this xref is called also at other places, do you get 
> more message or just the one above?
> 


Well, it turns out "nawk" is a link to "awk" (which is gawk) on my system.
But "mawk" gives the same message... investigating!


--
To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.


Re: [PATCH 07/23] doc: changed section for man page of "wdread" to 1x for consistency

2015-04-18 Thread Christophe

- Douglas Torrance  a écrit :
> On 04/06/2015 10:57 AM, Christophe CURIS wrote:
> > All the man pages for our tools that interact with Window Maker are placed
> > in the "1x" section, but the "wdread" page was an exception for no known
> > reason.
> >
> > For consistency, this patch renames the file to the same "1x" section.
> 
> Thanks for these patches, Christophe!
> 
> I'm wondering, though, if we should change all of the sections to 1 
> instead of 1x, as it appears deprecated.  Not even X manpages use 1x any 
> more.  On my system, "ls /usr/share/man/man1 | grep 1x" returns only 
> Window Maker-related manpages.


The situation seems to be similar on my side too.
Will make a patch about this, it makes sense!


--
To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.


Re: [PATCH 14/23] texi2txt: add support for making cross-references in the document

2015-04-18 Thread Christophe

- Douglas Torrance  a écrit :
> On 04/06/2015 10:58 AM, Christophe CURIS wrote:
> > The texinfo format provides 3 commands @ref, @xref and @pxref to make cross
> > references to existing @nodes in the document; it also provides a command
> > @anchor to place arbitrary targets for cross-reference.
> >
> > Because these will be handy for the Installation Manual that already does
> > some references, this patch implements the 4 commands:
> >
> > [...]
> 
> There must be some GNU extensions in here somewhere, as I get the 
> following error without gawk when running the script on Compilation.texi:
> 
> Error: cross reference to undefined node/anchor "ConfigureOptions" found 
> at line 227


Unfortunately I was not able to reproduce so far on #next; I tried with:
  awk --posix
  awk --traditional
  nawk

None of them is complaining. Looking at the message, it seems there is a space 
missing between the 2 words, so it looks like a difference in parsing. What's 
strange is that this xref is called also at other places, do you get more 
message or just the one above?


--
To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.


[PATCH 1/1] doc: re-generate the top directory documentation when "make dist" is invoked

2015-04-18 Thread Christophe CURIS
As pointed by Douglas Torrance, there are some case where the 2 docs
INSTALL-WMAKER and README.i18n are not re-generated properly, one of the
cases being if the user grabs the sources from Git to make a source package
only (ie, without compiling anything).

This patch adds a hook to "make dist" so it will re-generate the docs if
needed. Because the "dist-hook" is run after Automake has copied the files
to the temporary directory (distdir) then we also include an explicit copy
of the files there to have the latest version.

Signed-off-by: Christophe CURIS 
---
 doc/build/Makefile.am | 7 +++
 1 file changed, 7 insertions(+)

diff --git a/doc/build/Makefile.am b/doc/build/Makefile.am
index c80394e..dc656e9 100644
--- a/doc/build/Makefile.am
+++ b/doc/build/Makefile.am
@@ -7,6 +7,13 @@ EXTRA_DIST = Readme \
 # How to re-generate automatically the top-level text files
 all-local: $(top_srcdir)/INSTALL-WMAKER $(top_srcdir)/README.i18n
 
+# We also re-generate the documentation when "make dist" is used, because we 
cannot
+# be assured that the doc currently present in the directory is up-to-date, 
for example
+# if the user did not run "make (all)" for valid reason
+dist-hook: $(top_srcdir)/INSTALL-WMAKER $(top_srcdir)/README.i18n
+   cp -f $(top_srcdir)/INSTALL-WMAKER $(top_distdir)/INSTALL-WMAKER
+   cp -f $(top_srcdir)/README.i18n $(top_distdir)/README.i18n
+
 $(top_srcdir)/INSTALL-WMAKER: $(srcdir)/Compilation.texi 
$(top_srcdir)/script/generate-txt-from-texi.sh
$(AM_V_GEN)if test ! -e "$(top_srcdir)/INSTALL-WMAKER" -o -w 
"$(top_srcdir)/INSTALL-WMAKER" ; then \
$(top_srcdir)/script/generate-txt-from-texi.sh \
-- 
2.1.4


-- 
To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.


Re: [PATCH 15/23] doc: convert INSTALL-WMAKER into a texinfo source processed by texi2txt

2015-04-18 Thread Christophe

- Douglas Torrance  a écrit :
> 
> On 04/06/2015 10:58 AM, Christophe CURIS wrote:
> > diff --git a/doc/build/Makefile.am b/doc/build/Makefile.am
> > index 93e3e8b..c80394e 100644
> > --- a/doc/build/Makefile.am
> > +++ b/doc/build/Makefile.am
> > @@ -1,10 +1,20 @@
> >   # The list of sources are distributed, but none are to be
> >   # installed along with Window Maker:
> >   EXTRA_DIST = Readme \
> > +   Compilation.texi \
> > Translations.texi
> >   
> >   # How to re-generate automatically the top-level text files
> > -all-local: $(top_srcdir)/README.i18n
> > +all-local: $(top_srcdir)/INSTALL-WMAKER $(top_srcdir)/README.i18n
> > +
> > +$(top_srcdir)/INSTALL-WMAKER: $(srcdir)/Compilation.texi 
> > $(top_srcdir)/script/generate-txt-from-texi.sh
> > +   $(AM_V_GEN)if test ! -e "$(top_srcdir)/INSTALL-WMAKER" -o -w 
> > "$(top_srcdir)/INSTALL-WMAKER" ; then \
> > +   $(top_srcdir)/script/generate-txt-from-texi.sh \
> > +   $(srcdir)/Compilation.texi -o $(top_srcdir)/INSTALL-WMAKER \
> > +   -Dversion="$(PACKAGE_VERSION)" -e "$(PACKAGE_BUGREPORT)" ; \
> > +   else \
> > +   echo "Warning: \"$(top_srcdir)/INSTALL-WMAKER\" is not writeable, 
> > not regenerated" ; \
> > +   fi
> >   
> 
> Would it be possible for this to run during "make dist"?
> 
> Right now, the INSTALL-WMAKER in a "make dist"-generated tarball is the 
> one generated by autogen.sh and lists "git#next" as its version (the 
> default).  But when Window Maker is built, a new INSTALL-WMAKER is 
> generated with the version from configure.ac.
> 
> This can cause problems, e.g., when building Debian packages where the 
> source tree is expected to match the source tarball.  There's no problem 
> the first time, but the build fails on successive attempts.
> 
> This is also a problem for README.i18n.


Not really hard to do, and I think your case is totally valid.


Carlos,

Here's a patch to handle the issue.


--
To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.


[PATCH 3/3] merge with 418eb1f157c76ffc3db43f9a4fbf1a85f966d55c

2015-04-18 Thread Christophe CURIS
---
 doc/get-wraster-flags.1 | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/doc/get-wraster-flags.1 b/doc/get-wraster-flags.1
index 95bd062..8abc35f 100644
--- a/doc/get-wraster-flags.1
+++ b/doc/get-wraster-flags.1
@@ -4,11 +4,11 @@
 .PP
 .SH "SYNOPSIS"
 .B pkg-config
-.R wrlib
+wrlib
 .I \-\-cflags
 .br
 .B pkg-config
-.R wrlib
+wrlib
 .I \-\-libs
 .SH "DESCRIPTION"
 \fBget-wraster-flags\fP used to provide the compilation and link flags for 
programs that use
-- 
2.1.4


-- 
To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.


[PATCH 2/3] merge with 4c86603b36ed91da0b1246a4f0b4093d4e75bf08

2015-04-18 Thread Christophe CURIS
---
 doc/get-wutil-flags.1 | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/doc/get-wutil-flags.1 b/doc/get-wutil-flags.1
index 1e6641a..635c322 100644
--- a/doc/get-wutil-flags.1
+++ b/doc/get-wutil-flags.1
@@ -4,11 +4,11 @@
 .PP
 .SH "SYNOPSIS"
 .B pkg-config
-.R WUtil
+WUtil
 .I \-\-cflags
 .br
 .B pkg-config
-.R WUtil
+WUtil
 .I \-\-libs
 .SH "DESCRIPTION"
 \fBget-wutil-flags\fP used to provide the compilation and link flags for 
programs that use
-- 
2.1.4


-- 
To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.


[PATCH 1/3] merge with 46bc10e27b09cd7f9c149af982063fdf96a1a35f

2015-04-18 Thread Christophe CURIS
---
 doc/get-wings-flags.1 | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/doc/get-wings-flags.1 b/doc/get-wings-flags.1
index 7c04894..2ddd8b8 100644
--- a/doc/get-wings-flags.1
+++ b/doc/get-wings-flags.1
@@ -4,11 +4,11 @@
 .PP
 .SH "SYNOPSIS"
 .B pkg-config
-.R WINGs
+WINGs
 .I \-\-cflags
 .br
 .B pkg-config
-.R WINGs
+WINGs
 .I \-\-libs
 .SH "DESCRIPTION"
 \fBget-wings-flags\fP used to provide the compilation and link flags for 
programs that use
-- 
2.1.4


-- 
To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.


Re: [PATCH 03/23] WINGs: mark the script 'get-wings-flags' as deprecated

2015-04-18 Thread Christophe
Hi Douglas,

- Douglas Torrance  a écrit :
> Hi Christophe,
> 
> On 04/06/2015 10:57 AM, Christophe CURIS wrote:
> > --- a/doc/get-wings-flags.1
> > +++ b/doc/get-wings-flags.1
> > [...]
> > +.R WINGs
> 
> I'm getting the following warnings when I run "man --warnings 
> ./get-wings-flags.1 > /dev/null":
> 
> `R' is a string (producing the registered sign), not a macro.
> `R' is a string (producing the registered sign), not a macro.
> 
> This is true for all the get-*-flags manpages.

I confirm, I missed these because by default "man" does report anything and I 
did not know about this nice "--warnings" option...


Carlos,

I propose the following 3 patchs to fix that.

 doc/get-wings-flags.1   | 4 ++--
 doc/get-wraster-flags.1 | 4 ++--
 doc/get-wutil-flags.1   | 4 ++--
 3 files changed, 6 insertions(+), 6 deletions(-)


--
To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.