diff mbox

Bug in package tftp

Message ID 55D70AD3.2030302@gmx.de
State Not Applicable
Headers show

Commit Message

universe II Aug. 21, 2015, 11:26 a.m. UTC
Dear all,
when adding the tftp server to my target system image I found out that 
the tftp server is not started by the corresponding init script. Reason 
for that is a typo in the init script. The tftp binary is installed with 
the filename "tftpd" but the init script tries to call "in.tftpd". Just 
fixing the name in the script solves the problem. Please find below the 
patch for correcting the bug.

Regards,
Andreas

Comments

Vicente Olivert Riera Aug. 21, 2015, 1:29 p.m. UTC | #1
Dear Andreas,

On 08/21/2015 12:26 PM, universe II wrote:
> diff -Naur a/tftpd/S80tftpd-hpa b/tftpd/S80tftpd-hpa

that's not correct. You made the patch manually using "diff" from the
package directory, so the paths are not relative to $(TOPDIR). Anyway,
why you didn't make the patch using git?
You make the changes to the package/tftpd/S80tftpd-hpa file. Add that
file. Commit. And then use "git send-email --to=buildroot@buildroot.org
-1" (-1 to send only one patch starting from the HEAD).

Regards,

Vincent.
Peter Korsgaard Aug. 24, 2015, 3:16 p.m. UTC | #2
>>>>> "universe" == universe  <universeii@gmx.de> writes:

 > Dear all,
 > when adding the tftp server to my target system image I found out that
 > the tftp server is not started by the corresponding init
 > script. Reason for that is a typo in the init script. The tftp binary
 > is installed with the filename "tftpd" but the init script tries to
 > call "in.tftpd". Just fixing the name in the script solves the
 > problem. Please find below the patch for correcting the bug.

 > Regards,
 > Andreas

Fixed, thanks. In the future, please provide git formatted patches and
use your real name in From: and your Signed-off-by lines.
diff mbox

Patch

diff -Naur a/tftpd/S80tftpd-hpa b/tftpd/S80tftpd-hpa
--- a/tftpd/S80tftpd-hpa        2015-08-07 11:38:37.572148556 +0200
+++ b/tftpd/S80tftpd-hpa        2015-08-07 14:44:19.394469103 +0200
@@ -6,7 +6,7 @@ 

  PATH=/usr/local/sbin:/usr/local/bin:/sbin:/bin:/usr/sbin:/usr/bin
  DESC="HPA's tftpd"
-NAME=in.tftpd
+NAME=tftpd
  DAEMON=/usr/sbin/$NAME
  PIDFILE=/var/run/$NAME.pid
  SCRIPTNAME=/etc/init.d/S80tftpd-hpa