WelcomeWelcome | FAQFAQ | DownloadsDownloads | WikiWiki

Author Topic: busybox wget patch  (Read 1765 times)

Offline wysiwyg

  • Sr. Member
  • ****
  • Posts: 266
busybox wget patch
« on: July 20, 2016, 08:04:08 AM »
Good morning everyone, I was going to recompile the BB binaries to add some additional applets and noticed that there wasn't a patched version of the BB source code in the repo.  While applying the three patches, I noticed that the busybox-1.24.1_wget_3.patch has two parts of which the second does not seem applicable any longer.  This isn't critical, but does cause problems for automated processes since the patch fails.  Do you guys want the patched code to place in the repo like it was in the 6.x (and possibly earlier versions)?

Thanks,
Dave

Offline curaga

  • Administrator
  • Hero Member
  • *****
  • Posts: 10963
Re: busybox wget patch
« Reply #1 on: July 20, 2016, 10:36:57 AM »
It applies to today's busybox master?

patching file networking/Config.src
Hunk #1 succeeded at 967 with fuzz 2 (offset -16 lines).
patching file networking/wget.c
Hunk #1 succeeded at 1258 (offset 288 lines).

The second part is needed, and can't be dropped. Having patched bb source would be ok.
The only barriers that can stop you are the ones you create yourself.

Offline wysiwyg

  • Sr. Member
  • ****
  • Posts: 266
Re: busybox wget patch
« Reply #2 on: July 22, 2016, 08:19:09 AM »
Hey Curaga, when I applied the wget patch to the (unpatched) BB source downloaded from the TC repo, the second hunk failed.  After a visual inspection, I couldn't see that code to be patched was present at all which is why I said it doesn't appear to be used any longer:

$ cat networking/Config.src | grep 'G.timeout_seconds'
$

Thoughts?

Offline wysiwyg

  • Sr. Member
  • ****
  • Posts: 266
Re: busybox wget patch
« Reply #3 on: July 22, 2016, 08:26:23 AM »
I took a second look at this again and realized I was attempting to apply both hunks of the wget patch to the 'networking/Config.src' file, but the second hunk needs to be applied to 'networking/wget.c'.  Got it!  I'll send the patched source to you guys...

Dave