All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gustavo Zacarias <gustavo@zacarias.com.ar>
To: buildroot@busybox.net
Subject: [Buildroot] Deprecate rp-pppoe package
Date: Fri, 15 Oct 2010 17:25:38 -0300	[thread overview]
Message-ID: <4CB8B8C2.2010407@zacarias.com.ar> (raw)
In-Reply-To: <D61182AC8012EA4EBC531B3AF23BE1099C8F95@tranzeo-mail2.12stewart.tranzeo.com>

On 10/13/10 13:13, Darcy Watkins wrote:

> Hello,
> 
> I submitted a patch wrt this subject line / thread back in August.
> Essentially I updated the rp-pppoe package version (changed to
> autotools) and made sure that the PPPoE server side functionality works
> (on a powerpc target).
> 
> http://lists.busybox.net/pipermail/buildroot/2010-August/036907.html
> 
> http://lists.busybox.net/pipermail/buildroot/2010-July/036252.html
> 
> These changes were not included into buildroot-2010.08 because it was
> already at release candidate stage.  I don't see it committed / merged
> since either.  Will it make it in before 2010.11 hits release candidate
> stage?
> 
> 
> Regards,
> 
> Darcy

Hi Darcy.
Ok, managed to give it a spin on my powerpc rig.
My changes:

* RP_PPPOE_TARGETS_BIN and MANPAGES are basically the same, so merged.

* No need for RP_PPPOE_SOURCE if it uses the standard naming convention
(it does), so removed it.

* In INSTALL_TARGET_CMDS prefer $(INSTALL) over cp.

* NETSNMP_UNINSTALL? That should be RP_PPPOE_UNINSTALL :)

* In patch 002 (krn-plugin) you're hardcoding PLUGIN_DIR, i removed it
and just pass it to the package at build time (new RP_PPPOE_MAKE_OPTS)
based on pppd's package version. This is to avoid future breakage when
pppd changes version.

* Added RP_PPPOE_DEPENDENCIES = pppd, so that we build after pppd is
done, otherwise you may miss the pppd.h header.

* Cosmetic fixes.

Give it a spin, works for me.
Thanks and regards.
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: buildroot-rp-pppoe-3.10.patch
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20101015/e17c3916/attachment-0001.ksh>

  parent reply	other threads:[~2010-10-15 20:25 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-10-13 16:13 [Buildroot] Deprecate rp-pppoe package Darcy Watkins
2010-10-13 20:09 ` Gustavo Zacarias
2010-10-15 20:25 ` Gustavo Zacarias [this message]
  -- strict thread matches above, loose matches on Subject: below --
2010-07-06 17:08 Darcy L. Watkins
2010-07-20 18:47 ` Gustavo Zacarias
2010-07-20 19:44   ` Darcy Watkins
2010-06-01 19:51 Gustavo Zacarias

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4CB8B8C2.2010407@zacarias.com.ar \
    --to=gustavo@zacarias.com.ar \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.