r33629 - in /trunk/libnet-arp-perl/debian: changelog patches/10alignment-and-headers.patch patches/20skip_send_packet_test.patch patches/documentation.diff patches/get_mac.diff patches/prototypes.patch patches/return-value.patch

Mon Apr 20 22:24:20 UTC 2009

Author: thialme-guest
Date: Mon Apr 20 22:24:15 2009
New Revision: 33629

URL: http://svn.debian.org/wsvn/pkg-perl/?sc=1&rev=33629
Log:
Added header (author, bug number, description) to all patches.

Modified:
    trunk/libnet-arp-perl/debian/changelog
    trunk/libnet-arp-perl/debian/patches/10alignment-and-headers.patch
    trunk/libnet-arp-perl/debian/patches/20skip_send_packet_test.patch
    trunk/libnet-arp-perl/debian/patches/documentation.diff
    trunk/libnet-arp-perl/debian/patches/get_mac.diff
    trunk/libnet-arp-perl/debian/patches/prototypes.patch
    trunk/libnet-arp-perl/debian/patches/return-value.patch

Modified: trunk/libnet-arp-perl/debian/changelog
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/libnet-arp-perl/debian/changelog?rev=33629&op=diff
==============================================================================

--- trunk/libnet-arp-perl/debian/changelog (original)
+++ trunk/libnet-arp-perl/debian/changelog Mon Apr 20 22:24:15 2009
@@ -1,3 +1,9 @@
+libnet-arp-perl (1.0.3-3) unstable; urgency=low
+
+  * Added header (author, bug number, description) to all patches.
+
+ -- Franck Joncourt <franck.mail at dthconnex.com>  Tue, 21 Apr 2009 00:19:25 +0200
+
 libnet-arp-perl (1.0.3-2) unstable; urgency=low
 
   [ gregor herrmann ]

Modified: trunk/libnet-arp-perl/debian/patches/10alignment-and-headers.patch
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/libnet-arp-perl/debian/patches/10alignment-and-headers.patch?rev=33629&op=diff
==============================================================================
--- trunk/libnet-arp-perl/debian/patches/10alignment-and-headers.patch (original)
+++ trunk/libnet-arp-perl/debian/patches/10alignment-and-headers.patch Mon Apr 20 22:24:15 2009
@@ -1,3 +1,13 @@
+Author: Niko Tyni <ntyni at debian.org>
+Bugs: 390020, RT#25200
+Description: fix FTBFS due to unaligned assignment problems.
+ Added missing headers.
+ .
+ Defined our own structure my_arphdr, rather than using an existing name from
+ linux/if_arp.h which is different.
+ .
+ Do not assume arp->source_ip to be aligned, use memcpy instead. 
+
 --- a/ARP.xs
 +++ b/ARP.xs
 @@ -27,6 +27,7 @@ See the GNU General Public License for m

Modified: trunk/libnet-arp-perl/debian/patches/20skip_send_packet_test.patch
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/libnet-arp-perl/debian/patches/20skip_send_packet_test.patch?rev=33629&op=diff
==============================================================================
--- trunk/libnet-arp-perl/debian/patches/20skip_send_packet_test.patch (original)
+++ trunk/libnet-arp-perl/debian/patches/20skip_send_packet_test.patch Mon Apr 20 22:24:15 2009
@@ -1,3 +1,13 @@
+Author: Niko Tyni <ntyni at debian.org>, 
+ Franck Joncourt <franck.mail at dthconnex.com>
+Description: update tests for Debian 
+ The build dependency against Net:Pcap is dropped since it is only used to
+ find out an active network interface. The ifconfig(8) command is used instead
+ when DEBIAN_BUILD is set.
+ .
+ Sending ARP packets through the Net::ARP::send_packet function requires real
+ root privileges. Thus, we skip the send_packet test.
+
 --- libnet-arp-perl.orig/t/send_packet.t
 +++ libnet-arp-perl/t/send_packet.t
 @@ -5,7 +5,7 @@

Modified: trunk/libnet-arp-perl/debian/patches/documentation.diff
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/libnet-arp-perl/debian/patches/documentation.diff?rev=33629&op=diff
==============================================================================
--- trunk/libnet-arp-perl/debian/patches/documentation.diff (original)
+++ trunk/libnet-arp-perl/debian/patches/documentation.diff Mon Apr 20 22:24:15 2009
@@ -1,9 +1,8 @@
-From: Franck Joncourt <franck.mail at dthconnex.com>
-Subject: [PATCH] fixes/pod
-
-This patch fixes pod errors. (#524520)
-
-Signed-off-by: Franck Joncourt <franck.mail at dthconnex.com>
+Author: Franck Joncourt <franck.mail at dthconnex.com>
+Bugs: #524520, RT#45112
+Description: fix manpage errors
+ Add pod tags in order for *pod2man ARP.pm* to not display errors in the man
+ page of the module.
 
 ---
  ARP.pm |    4 ++++

Modified: trunk/libnet-arp-perl/debian/patches/get_mac.diff
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/libnet-arp-perl/debian/patches/get_mac.diff?rev=33629&op=diff
==============================================================================
--- trunk/libnet-arp-perl/debian/patches/get_mac.diff (original)
+++ trunk/libnet-arp-perl/debian/patches/get_mac.diff Mon Apr 20 22:24:15 2009
@@ -1,18 +1,16 @@
-From: Franck Joncourt <franck.mail at dthconnex.com>
-Subject: [PATCH] fixes/get_mac
-
-The get_mac function always returns "unknown" whatever interface is looked
-for.
-
-The device argument of the get_mac function must be defined out of the
-CODE scope in ARP.xs. The current behavior creates an undefined string which
-is passed to the get_mac_linux function.
-
-PS: the patch is kept as simple as possible, and based on what is currently
-working. More modifications would be required to make the get_mac function
-cleaner.
-
-Signed-off-by: Franck Joncourt <franck.mail at dthconnex.com>
+Author: Franck Joncourt <franck.mail at dthconnex.com>
+Bugs: 5524559, RT#45126
+Description: make the get_mac function used the user defined device
+ The result of the get_mac function is always *unknown* whatever interface
+ is looked for.
+ .
+ The device argument of the get_mac function must be defined out of the
+ CODE scope in ARP.xs. The current behavior creates an undefined string which
+ is passed to the get_mac_linux function.
+ .
+ PS: the patch is kept as simple as possible, and based on what is currently
+ working. More modifications would be required to make the get_mac function
+ cleaner.
 
 ---
  ARP.xs |    2 +-

Modified: trunk/libnet-arp-perl/debian/patches/prototypes.patch
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/libnet-arp-perl/debian/patches/prototypes.patch?rev=33629&op=diff
==============================================================================
--- trunk/libnet-arp-perl/debian/patches/prototypes.patch (original)
+++ trunk/libnet-arp-perl/debian/patches/prototypes.patch Mon Apr 20 22:24:15 2009
@@ -1,3 +1,8 @@
+Author: Niko Tyni <ntyni at debian.org>
+Description: add prototypes for the *_linux and *_bsd functions in arp.h
+ The arp header must define the global prototypes of the *_linux and *_bsd
+ functions used in order to let the ARP.xs file know about them.
+
 --- a/arp.h
 +++ b/arp.h
 @@ -56,3 +56,9 @@ struct my_arphdr {

Modified: trunk/libnet-arp-perl/debian/patches/return-value.patch
URL: http://svn.debian.org/wsvn/pkg-perl/trunk/libnet-arp-perl/debian/patches/return-value.patch?rev=33629&op=diff
==============================================================================
--- trunk/libnet-arp-perl/debian/patches/return-value.patch (original)
+++ trunk/libnet-arp-perl/debian/patches/return-value.patch Mon Apr 20 22:24:15 2009
@@ -1,4 +1,8 @@
-Add a missing return value at the end of a non-void function.
+Author: Niko Tyni <ntyni at debian.org>
+Description: add a missing return value at the end of a non-void function.
+ The arp_lookup_linux function currently returns -1 when an error occurs, but
+ do nothing when it succeeds.
+ This makes the function returns 0 rather than no value.
 
 --- a/arp_lookup_linux.c
 +++ b/arp_lookup_linux.c


