[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#500869: marked as done (Fixes for ocaml.mk)

Your message dated Tue, 03 Nov 2009 21:32:14 +0000
with message-id <E1N5Qz8-0004Eh-Hx@ries.debian.org>
and subject line Bug#500869: fixed in dh-ocaml 0.9.2
has caused the Debian Bug report #500869,
regarding Fixes for ocaml.mk
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
500869: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=500869
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: facile
Version: 1.1-6.1
Severity: normal
Tags: patch

        Hi there,

 I had a look at the facile source package today, and noticed some odd
 things in the packaging; I've cleaned it up a little in the attached
 debdiff.  (I'm filing this at normal severity because of the copyright
 issues.)

 Changelog:
  * Only compute OCAML_ABI once by using := instead of =.
  * Don't overwrite DEB_DH_GENCONTROL_ARGS and use -u instead of -- to pass
    args to dpkg-gencontrol in a safer manner.
  * Include dpatch.mk instead of calling dpatch manually; NB: this will cause
    the patches to be applied before running configure which is more useful,
    but doesn't change anything with the current patches; bump up cdbs bdep to
    >= 0.4.21.
  * Use the makefile.mk cdbs class instead of the autotools one.
    - Move the configure flag to a new custom common-configure-impl /
      config_Makefile rule.
    - Drop --disable-maintainer-mode exclusion hack.
    - Don't call make clean in clean; cdbs does that for us.
    - Set DEB_MAKE_INSTALL_TARGET to install.
  * Drop redundant setting of FACILEDIR with the echo >config_Makefile
    construct; the configure flag works fine.
  * Replace debian/libfacile-ocaml-dev.dirs.in template and logic with
    DEB_INSTALL_DIRS_ALL; fixes lack of cleanup of
    debian/libfacile-ocaml-dev.dirs.
  * Set OCAMLC OCAMLMLI via DEB_MAKE_INVOKE instead of DEB_MAKE_BUILD_TARGET;
    this sets the same environment for all make runs.
  * Add year 2004 to copyright.
  * Point at common-licenses for the full text of the LGPL and include the
    full licensing header.
  * Depend on ${misc:Depends} as recommended in debhelper 5.

 If you're happy with the changes, I can NMU this debdiff at your
 earliest convenience; woudln't hurt for lenny, but could as well be
 experimental or sit in usntable.

   Cheers,
-- 
Loïc Minier

diff -u facile-1.1/debian/rules facile-1.1/debian/rules
--- facile-1.1/debian/rules
+++ facile-1.1/debian/rules
@@ -3,20 +3,20 @@
-include /usr/share/cdbs/1/class/autotools.mk
 include /usr/share/cdbs/1/rules/debhelper.mk
+include /usr/share/cdbs/1/rules/dpatch.mk
+include /usr/share/cdbs/1/class/makefile.mk
 
-DEB_CONFIGURE_NORMAL_ARGS = --faciledir debian/libfacile-ocaml-dev/`ocamlc -where`/facile
-DEB_MAKE_BUILD_TARGET=OCAMLC="ocamlc -g" OCAMLMLI=ocamlc
-exclude=--disable-maintainer-mode
-DEB_CONFIGURE_INVOKE:=$(filter-out $(exclude),$(DEB_CONFIGURE_INVOKE))
-OCAMLABI = ${shell ocamlc -version}
-DEB_DH_GENCONTROL_ARGS=-- -VF:OCamlABI="$(OCAMLABI)"
+OCAMLABI := $(shell ocamlc -version)
+FACILEDIR := $(shell ocamlc -where)/facile
+DEB_INSTALL_DIRS_ALL += $(FACILEDIR)
+DEB_DH_GENCONTROL_ARGS += -u-VF:OCamlABI="$(OCAMLABI)"
+DEB_MAKE_INVOKE += OCAMLC="ocamlc -g" OCAMLMLI=ocamlc
+DEB_MAKE_INSTALL_TARGET := install
+
+common-configure-arch common-configure-indep:: common-configure-impl
+common-configure-impl:: config_Makefile
+config_Makefile:
+	./configure --faciledir debian/libfacile-ocaml-dev/$(FACILEDIR)
 
-common-configure-arch::  
-	echo FACILEDIR=debian/libfacile-ocaml-dev/`ocamlc -where`/facile >config_Makefile
-	sed -e 's%#OcamlABI#%$(OCAMLABI)%' debian/libfacile-ocaml-dev.dirs.in >debian/libfacile-ocaml-dev.dirs
-	dpatch apply-all
-	
 clean::
-	make clean
+	# not cleaned upstream
 	rm -f config_Makefile
-	dpatch deapply-all
-	rm -rf debian/patched
+
diff -u facile-1.1/debian/changelog facile-1.1/debian/changelog
--- facile-1.1/debian/changelog
+++ facile-1.1/debian/changelog
@@ -1,3 +1,32 @@
+facile (1.1-6.2) UNRELEASED; urgency=low
+
+  * Only compute OCAML_ABI once by using := instead of =.
+  * Don't overwrite DEB_DH_GENCONTROL_ARGS and use -u instead of -- to pass
+    args to dpkg-gencontrol in a safer manner.
+  * Include dpatch.mk instead of calling dpatch manually; NB: this will cause
+    the patches to be applied before running configure which is more useful,
+    but doesn't change anything with the current patches; bump up cdbs bdep to
+    >= 0.4.21.
+  * Use the makefile.mk cdbs class instead of the autotools one.
+    - Move the configure flag to a new custom common-configure-impl /
+      config_Makefile rule.
+    - Drop --disable-maintainer-mode exclusion hack.
+    - Don't call make clean in clean; cdbs does that for us.
+    - Set DEB_MAKE_INSTALL_TARGET to install.
+  * Drop redundant setting of FACILEDIR with the echo >config_Makefile
+    construct; the configure flag works fine.
+  * Replace debian/libfacile-ocaml-dev.dirs.in template and logic with
+    DEB_INSTALL_DIRS_ALL; fixes lack of cleanup of
+    debian/libfacile-ocaml-dev.dirs.
+  * Set OCAMLC OCAMLMLI via DEB_MAKE_INVOKE instead of DEB_MAKE_BUILD_TARGET;
+    this sets the same environment for all make runs.
+  * Add year 2004 to copyright.
+  * Point at common-licenses for the full text of the LGPL and include the
+    full licensing header.
+  * Depend on ${misc:Depends} as recommended in debhelper 5.
+
+ -- Loic Minier <lool@dooz.org>  Wed, 01 Oct 2008 21:17:11 +0200
+
 facile (1.1-6.1) unstable; urgency=low
 
   * Non-maintainer upload.
diff -u facile-1.1/debian/copyright facile-1.1/debian/copyright
--- facile-1.1/debian/copyright
+++ facile-1.1/debian/copyright
@@ -6,14 +6,20 @@
-Copyright Holder: Copyright (C) 2001, CENA 
+Copyright Holder: Copyright (C) 2001, 2004, CENA
 
 License:
 
-This library is free software; you can redistribute it and/or
-modify it under the terms of the GNU Lesser General Public
-License as published by the Free Software Foundation; either
-version 2.1 of the License, or (at your option) any later version.
-
-This library is distributed in the hope that it will be useful,
-but WITHOUT ANY WARRANTY; without even the implied warranty of
-MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
-Lesser General Public License for more details.
+    This library is free software; you can redistribute it and/or
+    modify it under the terms of the GNU Lesser General Public
+    License as published by the Free Software Foundation; either
+    version 2.1 of the License, or (at your option) any later version.
+
+    This library is distributed in the hope that it will be useful,
+    but WITHOUT ANY WARRANTY; without even the implied warranty of
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+    Lesser General Public License for more details.
+
+    You should have received a copy of the GNU Lesser General Public
+    License along with this package; if not, write to the Free Software
+    Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301 USA
 
+On Debian systems, the complete text of the GNU Lesser General
+Public License can be found in `/usr/share/common-licenses/LGPL'.
diff -u facile-1.1/debian/control facile-1.1/debian/control
--- facile-1.1/debian/control
+++ facile-1.1/debian/control
@@ -2,7 +2,7 @@
 Priority: optional
 Maintainer: Debian OCaml Maintainers <debian-ocaml-maint@lists.debian.org>
 Uploaders: Steffen Joeris <white@debian.org>
-Build-Depends: debhelper (>= 5), cdbs (>= 0.4.0), ocaml-nox (>= 3.09.2), dpatch
+Build-Depends: debhelper (>= 5), cdbs (>= 0.4.21), ocaml-nox (>= 3.09.2), dpatch
 Standards-Version: 3.7.2
 Section: libdevel
 XS-Vcs-Svn: svn://svn.debian.org/svn/pkg-ocaml-maint/trunk/packages/facile
@@ -11,7 +11,7 @@
 Package: libfacile-ocaml-dev
 Section: libdevel
 Architecture: any
-Depends: ocaml-nox-${F:OCamlABI} 
+Depends: ocaml-nox-${F:OCamlABI}, ${misc:Depends}
 Description: functional constraint library implemented in objective caml
  FaCiLe is a constraint programming library over integer finite domain
  written in Objective Caml. It offers all usual facilities to create
reverted:
--- facile-1.1/debian/libfacile-ocaml-dev.dirs.in
+++ facile-1.1.orig/debian/libfacile-ocaml-dev.dirs.in
@@ -1 +0,0 @@
-usr/lib/ocaml/#OcamlABI#/facile

--- End Message ---
--- Begin Message --- 
Source: dh-ocaml
Source-Version: 0.9.2

We believe that the bug you reported is fixed in the latest version of
dh-ocaml, which is due to be installed in the Debian FTP archive:

dh-ocaml_0.9.2.dsc
  to main/d/dh-ocaml/dh-ocaml_0.9.2.dsc
dh-ocaml_0.9.2.tar.gz
  to main/d/dh-ocaml/dh-ocaml_0.9.2.tar.gz
dh-ocaml_0.9.2_all.deb
  to main/d/dh-ocaml/dh-ocaml_0.9.2_all.deb



A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 500869@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Stéphane Glondu <glondu@debian.org> (supplier of updated dh-ocaml package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Format: 1.8
Date: Tue, 03 Nov 2009 20:35:55 +0100
Source: dh-ocaml
Binary: dh-ocaml
Architecture: source all
Version: 0.9.2
Distribution: unstable
Urgency: low
Maintainer: Debian OCaml Maintainers <debian-ocaml-maint@lists.debian.org>
Changed-By: Stéphane Glondu <glondu@debian.org>
Description: 
 dh-ocaml   - helper tools for maintaining OCaml-related Debian packages
Closes: 500869 552392
Changes: 
 dh-ocaml (0.9.2) unstable; urgency=low
 .
   [ Stéphane Glondu ]
   * dom-new-git-repo: fail by default when executed out of a Git
     repository and without tarball argument; don't fail if explicitly
     given "--notifications"
   * dh_ocaml: also take into account OCaml objects in non-lib packages;
     don't assume all objects are in /usr/lib
   * ocaml.mk: don't overwrite DEB_DH_GENCONTROL_ARGS and use -u instead of
     -- to pass args to dpkg-gencontrol in a safer manner (thanks to Loïc
     Minier) (Closes: #500869)
 .
   [ Mehdi Dogguy ]
   * Use my debian address instead of the old ones, everywhere.
   * Handle relations between libXXXX-camlp4-dev and libXXXX-camlp4
     packages (Closes: #552392).
   * dh_ocamlinit: Do not generate a stamp file after cleaning
   * ocaml.mk: run dh_ocamldoc only on the current package
   * OCAML_OCAMLDOC_PACKAGES is not needed anymore. Thus, removing each
     occurence and reference to it.
 .
   [ Samuel Mimram ]
   * dom-git-checkout: add --no-ssh option
Checksums-Sha1: 
 5a1352fbb1e6fb60906c52c2333a87d6f215db59 1771 dh-ocaml_0.9.2.dsc
 419f6441aef7a0a7c3109578183f9bb450ea87ca 49599 dh-ocaml_0.9.2.tar.gz
 f5f44b9ae84cfffbc15b3678c3952895da71c952 80536 dh-ocaml_0.9.2_all.deb
Checksums-Sha256: 
 7063d115ac02911be806459cf861b09a249f9a0084bbb1b6f2b3a81f959206b5 1771 dh-ocaml_0.9.2.dsc
 7bc6b7876245aa77da02703252d52f20424b4854e9efc596015fe77e383b0803 49599 dh-ocaml_0.9.2.tar.gz
 a981e0bb40edd1342047ee2ea717030898bd87da270175069ac0eb6dc8f67634 80536 dh-ocaml_0.9.2_all.deb
Files: 
 0746889aa7cca4e014e90f74f82212ad 1771 ocaml extra dh-ocaml_0.9.2.dsc
 2661b4bfaf4dc44a871cae07d9d37c15 49599 ocaml extra dh-ocaml_0.9.2.tar.gz
 4d23d3b67a6c94becddccecaf0fb306e 80536 ocaml extra dh-ocaml_0.9.2_all.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)

iQIcBAEBCgAGBQJK8IaOAAoJEHhT2k1JiBrTI00P/3+JeQzurByJpeaUchWTI/6G
qQqgMFHpkcU7JI+4AViqeZU/zeo2DkSA4WqY8QnbZUJecwLcxkAbycbksTThOLli
p3e4qRfEdnAte8X5kdPTtrTl6GZoIjz0n2r3wLg5sBai5Wq7ruMGM0k3R9qFGYr6
Hu8mLGuR/hZvVkqC5/xhkeZXcSBj4C1L2Zy3w/a5H2OPTI+8ArsVbVV2Hul1FUOh
5VJWQlmFifUcmXQSDvOj4JXI6HDzSIiva17gBfIcQAJll6q216Kd/l/NKno7dysd
PchXFYULbSayt6NWKu6t9zze8b/ZjaHDeFB8OP2X4qe43q8M37W0K89lK57q+FtE
WZQVwT/sEz9D9rWQPYBgTRnSNtPUMmgnjX/f57gfgRCmtS6Lc4GGcWbINQ9PIKCR
BvoloXg/J/BxnzNkNeRbwbH2Tde29V2MAAEGFNj4M+MiYOfM4j0BkdxoRrvpQOlu
1xWEMWFtinKg8NpypNNzUfEwHYOWundLggKdNb6xMoAjEKKDAF1jfyqP6XNOY3tR
qfvO2eDlg+lWSHgOBwwfPZJmV/7PEK1k3am+Qxr85P6NY4VKqsHxPDXnJTdA2o95
XqNcg93QqMG5MIcuwA2tKtluKUbU+p6mm2oPmbdqXQIwCxK9rtXE9FfNYvY3nUZM
+4woxzHuVPf8sbs0ri32
=Xt70
-----END PGP SIGNATURE-----

--- End Message ---
Reply to: