Hello community,

here is the log from the commit of package yast2-printer for openSUSE:Factory 
checked in at 2013-09-14 19:08:06
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/yast2-printer (Old)
 and      /work/SRC/openSUSE:Factory/.yast2-printer.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "yast2-printer"

Changes:
--------
--- /work/SRC/openSUSE:Factory/yast2-printer/yast2-printer.changes      
2013-08-02 15:47:30.000000000 +0200
+++ /work/SRC/openSUSE:Factory/.yast2-printer.new/yast2-printer.changes 
2013-09-14 19:08:08.000000000 +0200
@@ -1,0 +2,17 @@
+Fri Sep 13 12:08:29 CEST 2013 - jsm...@suse.de
+
+- test_if_hplip_is_installed.diff is a simple band-aid patch
+  to avoid that the AddDriverDialog works inconsistently
+  regarding HPLIP RPMs (see bnc#837770).
+  The real solution will be implemented later according
+  to what is described in bnc#808315 comment#3.
+
+-------------------------------------------------------------------
+Wed Sep 11 10:58:19 CEST 2013 - jsm...@suse.de
+
+- Using newest yast2-printer-3.0.0.tar.bz2 from YaST:Head
+  that differs only in autodocs-ycp.ami configure configure.ac
+  doc/autodocs/Makefile.in Makefile.am.common src/Makefile.in
+  (i.e. no changes in the actual program's source code files).
+
+-------------------------------------------------------------------

New:
----
  test_if_hplip_is_installed.diff

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ yast2-printer.spec ++++++
--- /var/tmp/diff_new_pack.6gY17n/_old  2013-09-14 19:08:09.000000000 +0200
+++ /var/tmp/diff_new_pack.6gY17n/_new  2013-09-14 19:08:09.000000000 +0200
@@ -23,6 +23,14 @@
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build
 Source0:        yast2-printer-%{version}.tar.bz2
 
+# Patch0 test_if_hplip_is_installed.diff is a simple band-aid patch to avoid 
that the
+# AddDriverDialog works inconsistently regarding HPLIP RPMs (hplip-hpijs 
versus hplip)
+# see https://bugzilla.novell.com/show_bug.cgi?id=837770
+# and restore consistent behaviour and sufficiently consistent texts.
+# The real solution will be implemented later according to what is described
+# in https://bugzilla.novell.com/show_bug.cgi?id=808315#c3
+Patch0:         test_if_hplip_is_installed.diff
+
 BuildRequires:  update-desktop-files
 BuildRequires:  xorg-x11-libX11-devel
 BuildRequires:  yast2
@@ -61,6 +69,11 @@
 
 %prep
 %setup -n yast2-printer-%{version}
+# Patch0 test_if_hplip_is_installed.diff is a simple band-aid patch to avoid 
that the
+# AddDriverDialog works inconsistently regarding HPLIP RPMs (hplip-hpijs 
versus hplip)
+# see https://bugzilla.novell.com/show_bug.cgi?id=837770
+# and restore consistent behaviour and sufficiently consistent texts:
+%patch0
 
 %build
 %{_prefix}/bin/y2tool y2autoconf

++++++ test_if_hplip_is_installed.diff ++++++
--- src/include/printer/driveradd.rb.orig       2013-07-30 13:39:28.000000000 
+0200
+++ src/include/printer/driveradd.rb    2013-09-13 11:59:59.000000000 +0200
@@ -62,7 +62,7 @@ module Yast
         "installed"
       )
       hplip_installed = Printerlib.TestAndInstallPackage(
-        "hplip-hpijs",
+        "hplip",
         "installed"
       )
       manufacturer_PPDs_installed = Printerlib.TestAndInstallPackage(
++++++ yast2-printer-3.0.0.tar.bz2 ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-printer-3.0.0/Makefile.am.common 
new/yast2-printer-3.0.0/Makefile.am.common
--- old/yast2-printer-3.0.0/Makefile.am.common  2013-07-31 12:21:12.000000000 
+0200
+++ new/yast2-printer-3.0.0/Makefile.am.common  2013-08-13 16:11:31.000000000 
+0200
@@ -15,6 +15,7 @@
 CLEANFILES = ${ybcfiles}
 DISTCLEANFILES = .dep
 
+if HAS_YCP_MODULES
 # Needs to be outside "." because of cases
 # where ycpchook contains a symlink to "."
 # Otherwise "." keeps being newer than .dep and we loop.
@@ -26,14 +27,16 @@
        ${YCPMAKEDEP} --prefix=${prefix} ${YCPCFLAGS} > ${NEWDEP}
        cat ${NEWDEP} > .dep
        rm ${NEWDEP}
+-include .dep
+endif
 
 dist-hook: check-syntax
 
 CHECK_SYNTAX = true
-check-syntax: $(client_DATA) $(ycpchook) $(ybcfiles)
-       @if test "$(filter %.ycp,client_DATA)"; then \
+check-syntax: $(client_DATA) $(module_DATA) $(ynclude_DATA) $(ydata_DATA) 
$(ycpchook) $(ybcfiles)
+       @if test "$(filter %.ycp, $(client_DATA) $(ydata_DATA))"; then \
          if $(CHECK_SYNTAX); then \
-           TO_CHECK="$(filter %.ycp,$^)"; \
+           TO_CHECK="$(filter %.ycp, $(client_DATA) $(ydata_DATA))"; \
            echo "Checking syntax of $${TO_CHECK}"; \
            if ! $(YCPC) -qE -M. -I. $(YCPCFLAGS) $${TO_CHECK}; then \
              echo "Failed. Disable by 'make ... CHECK_SYNTAX=false' at your 
own risk"; \
@@ -43,5 +46,18 @@
            echo "Syntax check disabled"; \
          fi; \
        fi
+       @if test "$(filter %.rb, $^)"; then \
+         if $(CHECK_SYNTAX); then \
+           TO_CHECK="$(filter %.rb, $^)"; \
+           echo "Checking syntax of $${TO_CHECK}"; \
+           for f in $${TO_CHECK}; do \
+             if ! ruby -c $$f > /dev/null; then \
+               echo "Failed. Disable by 'make ... CHECK_SYNTAX=false' at your 
own risk"; \
+               exit 1; \
+             fi; \
+           done; \
+         else \
+           echo "Syntax check disabled"; \
+         fi; \
+       fi
 
--include .dep
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-printer-3.0.0/autodocs-ycp.ami 
new/yast2-printer-3.0.0/autodocs-ycp.ami
--- old/yast2-printer-3.0.0/autodocs-ycp.ami    2013-07-31 12:21:12.000000000 
+0200
+++ new/yast2-printer-3.0.0/autodocs-ycp.ami    2013-08-13 16:11:31.000000000 
+0200
@@ -17,8 +17,11 @@
 
 htmldir = $(docdir)/$(AUTODOCS_SUBDIR)/autodocs
 
+# find all files in Yast/ subdirectory if it exists
+YARD_YAST_FILES = $(shell test -d Yast && find Yast -type f)
+
 # use nobase_ prefix to keep the directory structure
-nobase_html_DATA = index.html $(filter-out index.html, $(wildcard *.html)) 
$(wildcard js/*) $(wildcard css/*) $(wildcard Yast/*)
+nobase_html_DATA = index.html $(filter-out index.html, $(wildcard *.html)) 
$(wildcard js/*) $(wildcard css/*) $(YARD_YAST_FILES)
 CLEANFILES = $(nobase_html_DATA) pod2htm*.tmp
 
 AUTODOCS_YCP ?= $(wildcard $(srcdir)/../../src/*.ycp)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-printer-3.0.0/configure 
new/yast2-printer-3.0.0/configure
--- old/yast2-printer-3.0.0/configure   2013-07-31 12:21:16.000000000 +0200
+++ new/yast2-printer-3.0.0/configure   2013-08-13 16:11:35.000000000 +0200
@@ -627,6 +627,8 @@
 am__EXEEXT_TRUE
 LTLIBOBJS
 LIBOBJS
+HAS_YCP_MODULES_FALSE
+HAS_YCP_MODULES_TRUE
 YCPC
 YCPMAKEDEP
 YCPDOC
@@ -4553,6 +4555,29 @@
 fi
 
 
+# handle .dep files in Makefile.am.common if any YCP module is present
+{ $as_echo "$as_me:${as_lineno-$LINENO}: checking for YCP modules" >&5
+$as_echo_n "checking for YCP modules... " >&6; }
+
+# YCP module file name starts with an upper case letter
+find . -type f | grep -q "[[:upper:]][^/]*\.ycp$" && has_ycp_modules=1
+ if test -n "$has_ycp_modules"; then
+  HAS_YCP_MODULES_TRUE=
+  HAS_YCP_MODULES_FALSE='#'
+else
+  HAS_YCP_MODULES_TRUE='#'
+  HAS_YCP_MODULES_FALSE=
+fi
+
+
+if test -n "$has_ycp_modules"; then
+  { $as_echo "$as_me:${as_lineno-$LINENO}: result: found" >&5
+$as_echo "found" >&6; }
+else
+  { $as_echo "$as_me:${as_lineno-$LINENO}: result: not found" >&5
+$as_echo "not found" >&6; }
+fi
+
 
 ac_config_files="$ac_config_files Makefile data/Makefile doc/autodocs/Makefile 
doc/Makefile src/Makefile testsuite/Makefile tools/Makefile"
 
@@ -4733,6 +4758,10 @@
   as_fn_error $? "conditional \"CREATE_PKGCONFIG_NOARCH\" was never defined.
 Usually this means the macro was only invoked conditionally." "$LINENO" 5
 fi
+if test -z "${HAS_YCP_MODULES_TRUE}" && test -z "${HAS_YCP_MODULES_FALSE}"; 
then
+  as_fn_error $? "conditional \"HAS_YCP_MODULES\" was never defined.
+Usually this means the macro was only invoked conditionally." "$LINENO" 5
+fi
 
 : "${CONFIG_STATUS=./config.status}"
 ac_write_fail=0
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-printer-3.0.0/configure.ac 
new/yast2-printer-3.0.0/configure.ac
--- old/yast2-printer-3.0.0/configure.ac        2013-07-31 12:21:12.000000000 
+0200
+++ new/yast2-printer-3.0.0/configure.ac        2013-08-13 16:11:31.000000000 
+0200
@@ -1,6 +1,6 @@
 dnl configure.ac for yast2-printer
 dnl
-dnl -- This file is generated by y2autoconf 2.24.7 - DO NOT EDIT! --
+dnl -- This file is generated by y2autoconf 3.0.4 - DO NOT EDIT! --
 dnl    (edit configure.ac.in or configure.in.in instead)
 
 AC_INIT(yast2-printer, 3.0.0, http://bugs.opensuse.org/, yast2-printer)
@@ -156,6 +156,19 @@
     AC_MSG_WARN([yast2-testsuite.rpm is not installed])
 ])
 
+# handle .dep files in Makefile.am.common if any YCP module is present
+AC_MSG_CHECKING([for YCP modules])
+
+# YCP module file name starts with an upper case letter
+[find . -type f | grep -q "[[:upper:]][^/]*\.ycp$" && has_ycp_modules=1]
+AM_CONDITIONAL([HAS_YCP_MODULES], [test -n "$has_ycp_modules"])
+
+if test -n "$has_ycp_modules"; then
+  AC_MSG_RESULT([found])
+else
+  AC_MSG_RESULT([not found])
+fi
+
 
 AC_CONFIG_FILES(Makefile data/Makefile
 doc/autodocs/Makefile
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-printer-3.0.0/doc/autodocs/Makefile.in 
new/yast2-printer-3.0.0/doc/autodocs/Makefile.in
--- old/yast2-printer-3.0.0/doc/autodocs/Makefile.in    2013-07-31 
12:21:17.000000000 +0200
+++ new/yast2-printer-3.0.0/doc/autodocs/Makefile.in    2013-08-13 
16:11:36.000000000 +0200
@@ -249,8 +249,11 @@
 yncludedir = @yncludedir@
 ystartupdir = @ystartupdir@
 
+# find all files in Yast/ subdirectory if it exists
+YARD_YAST_FILES = $(shell test -d Yast && find Yast -type f)
+
 # use nobase_ prefix to keep the directory structure
-nobase_html_DATA = index.html $(filter-out index.html, $(wildcard *.html)) 
$(wildcard js/*) $(wildcard css/*) $(wildcard Yast/*)
+nobase_html_DATA = index.html $(filter-out index.html, $(wildcard *.html)) 
$(wildcard js/*) $(wildcard css/*) $(YARD_YAST_FILES)
 CLEANFILES = $(nobase_html_DATA) pod2htm*.tmp
 all: all-am
 
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-printer-3.0.0/src/Makefile.in 
new/yast2-printer-3.0.0/src/Makefile.in
--- old/yast2-printer-3.0.0/src/Makefile.in     2013-07-31 12:21:17.000000000 
+0200
+++ new/yast2-printer-3.0.0/src/Makefile.in     2013-08-13 16:11:36.000000000 
+0200
@@ -281,7 +281,7 @@
 # Needs to be outside "." because of cases
 # where ycpchook contains a symlink to "."
 # Otherwise "." keeps being newer than .dep and we loop.
-NEWDEP = ${top_builddir}/.dep.new
+@HAS_YCP_MODULES_TRUE@NEWDEP = ${top_builddir}/.dep.new
 CHECK_SYNTAX = true
 all: all-am
 
@@ -620,16 +620,17 @@
 # generate dependencies
 # two steps not to lose the file if the command fails
 # hook: create the links before we look for files
-.dep: $(wildcard *.ycp) ${ycpchook}
-       ${YCPMAKEDEP} --prefix=${prefix} ${YCPCFLAGS} > ${NEWDEP}
-       cat ${NEWDEP} > .dep
-       rm ${NEWDEP}
+@HAS_YCP_MODULES_TRUE@.dep: $(wildcard *.ycp) ${ycpchook}
+@HAS_YCP_MODULES_TRUE@ ${YCPMAKEDEP} --prefix=${prefix} ${YCPCFLAGS} > 
${NEWDEP}
+@HAS_YCP_MODULES_TRUE@ cat ${NEWDEP} > .dep
+@HAS_YCP_MODULES_TRUE@ rm ${NEWDEP}
+@HAS_YCP_MODULES_TRUE@-include .dep
 
 dist-hook: check-syntax
-check-syntax: $(client_DATA) $(ycpchook) $(ybcfiles)
-       @if test "$(filter %.ycp,client_DATA)"; then \
+check-syntax: $(client_DATA) $(module_DATA) $(ynclude_DATA) $(ydata_DATA) 
$(ycpchook) $(ybcfiles)
+       @if test "$(filter %.ycp, $(client_DATA) $(ydata_DATA))"; then \
          if $(CHECK_SYNTAX); then \
-           TO_CHECK="$(filter %.ycp,$^)"; \
+           TO_CHECK="$(filter %.ycp, $(client_DATA) $(ydata_DATA))"; \
            echo "Checking syntax of $${TO_CHECK}"; \
            if ! $(YCPC) -qE -M. -I. $(YCPCFLAGS) $${TO_CHECK}; then \
              echo "Failed. Disable by 'make ... CHECK_SYNTAX=false' at your 
own risk"; \
@@ -639,8 +640,20 @@
            echo "Syntax check disabled"; \
          fi; \
        fi
-
--include .dep
+       @if test "$(filter %.rb, $^)"; then \
+         if $(CHECK_SYNTAX); then \
+           TO_CHECK="$(filter %.rb, $^)"; \
+           echo "Checking syntax of $${TO_CHECK}"; \
+           for f in $${TO_CHECK}; do \
+             if ! ruby -c $$f > /dev/null; then \
+               echo "Failed. Disable by 'make ... CHECK_SYNTAX=false' at your 
own risk"; \
+               exit 1; \
+             fi; \
+           done; \
+         else \
+           echo "Syntax check disabled"; \
+         fi; \
+       fi
 
 # Tell versions [3.59,3.63) of GNU make to not export all variables.
 # Otherwise a system limit (for SysV at least) may be exceeded.

-- 
To unsubscribe, e-mail: opensuse-commit+unsubscr...@opensuse.org
For additional commands, e-mail: opensuse-commit+h...@opensuse.org

Reply via email to