r76075 - in /trunk/libnet-whois-parser-perl/debian: changelog patches/noInternet.patch rules

jotamjr-guest at users.alioth.debian.org jotamjr-guest at users.alioth.debian.org
Sun Jun 19 00:58:30 UTC 2011


Author: jotamjr-guest
Date: Sun Jun 19 00:58:28 2011
New Revision: 76075

URL: http://svn.debian.org/wsvn/pkg-perl/?sc=1&rev=76075
Log:
* debian/rules: use "dh_auto_test" instead of "$(MAKE) test"
  in dh_auto_test override
* debian/patches/noInternet.patch: drop useless test case
  for environment variable

Modified:
    trunk/libnet-whois-parser-perl/debian/changelog
    trunk/libnet-whois-parser-perl/debian/patches/noInternet.patch
    trunk/libnet-whois-parser-perl/debian/rules

Modified: trunk/libnet-whois-parser-perl/debian/changelog
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/libnet-whois-parser-perl/debian/changelog?rev=76075&op=diff
==============================================================================
--- trunk/libnet-whois-parser-perl/debian/changelog (original)
+++ trunk/libnet-whois-parser-perl/debian/changelog Sun Jun 19 00:58:28 2011
@@ -1,13 +1,4 @@
 libnet-whois-parser-perl (0.05-3) UNRELEASED; urgency=low
-
-  TODO:
-  - nothing dramatic, just a few thoughts about the tests/internet:
-    + in debian/rules you can use "connected_to_net=0 dh_auto_test"
-    + often environment variables use CAPITAL LETTERS
-    + I don't think the "if(defined $ENV{connected_to_net})" is needed in the
-      patch
-  - d/control: "perl (>=5.11.3) | libmodule-build-perl (>=0.360000)" belongs
-    in Build-Depends instead of B-D-I (needed during clean)
 
   * debian/control: add missing dependency (libnet-whois-raw-perl), thanks
     to Antti Salmela <asalmela at iki.fi> for reporting the bug,
@@ -21,6 +12,10 @@
   * debian/copyright: add year to upstream copyright statement
   * debian/control: change maintainer to pkg-perl group
   * debian/control: add Vcs-Svn and Vcs-Browser fields
+  * debian/rules: use "dh_auto_test" instead of "$(MAKE) test"
+    in dh_auto_test override
+  * debian/patches/noInternet.patch: drop useless test case
+    for environment variable
 
  -- Jotam Jr. Trejo <jotamjr at debian.org.sv>  Sat, 18 Jun 2011 17:14:20 -0600
 

Modified: trunk/libnet-whois-parser-perl/debian/patches/noInternet.patch
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/libnet-whois-parser-perl/debian/patches/noInternet.patch?rev=76075&op=diff
==============================================================================
--- trunk/libnet-whois-parser-perl/debian/patches/noInternet.patch (original)
+++ trunk/libnet-whois-parser-perl/debian/patches/noInternet.patch Sun Jun 19 00:58:28 2011
@@ -5,21 +5,19 @@
 Forwarded: not-needed
 Bug-Debian: http://bugs.debian.org/577350
 Author: Jotam Jr. Trejo <jotamjr at debian.org.sv>
-Last-Update: 2010-04-14
+Last-Update: 2011-06-18
 
 --- a/t/000-base.t
 +++ b/t/000-base.t
-@@ -14,7 +14,10 @@ my $domain = 'reg.ru';
+@@ -14,6 +14,9 @@ my $domain = 'reg.ru';
  my $info;
  
  plan tests => 11;
--
 +SKIP: {
-+	  if(defined $ENV{connected_to_net}){
-+	      skip "Build system don't have internet access", 11 unless $ENV{connected_to_net};
-+	  }
++
++skip "Build system don't have internet access", 11 unless $ENV{CONNECTED_TO_NET};
+ 
  my ( $raw, $server ) = whois($domain);
- 
  
 @@ -52,4 +55,5 @@ $info = parse_whois( raw => $raw );
  

Modified: trunk/libnet-whois-parser-perl/debian/rules
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/libnet-whois-parser-perl/debian/rules?rev=76075&op=diff
==============================================================================
--- trunk/libnet-whois-parser-perl/debian/rules (original)
+++ trunk/libnet-whois-parser-perl/debian/rules Sun Jun 19 00:58:28 2011
@@ -4,7 +4,7 @@
 	dh $@
 
 override_dh_auto_test:
-	#use connected_to_net=1 when you have access to internet
+	#use CONNECTED_TO_NET=1 when you have access to internet
 	#this is set to 0 by defautl because debian build systems don't have
 	#internet access.
-	connected_to_net=0 $(MAKE) test
+	CONNECTED_TO_NET=0 dh_auto_test




More information about the Pkg-perl-cvs-commits mailing list