[oe] [meta-networking][PATCH] radiusclient-ng : add new recipe
Martin Jansa
martin.jansa at gmail.com
Wed Dec 3 17:54:14 UTC 2014
On Tue, Dec 02, 2014 at 03:45:04PM -0500, Joe MacDonald wrote:
> Hi Li,
>
> [[oe] [meta-networking][PATCH] radiusclient-ng : add new recipe] On 14.12.02 (Tue 11:19) Li xin wrote:
>
> > RADIUS protocol client library.
> > It is Portable, easy-to-use and standard compliant library suitable
> > for developing free and commercial software that need support for a RADIUS
> > protocol (RFCs 2128 and 2139).
> >
> > Signed-off-by: Li Xin <lixin.fnst at cn.fujitsu.com>
> > ---
> > .../files/Fix-configure-and-compile-errors.patch | 79 ++++++++++++++++++++++
> > .../files/config-site.radiusclient-ng-0.5.6 | 1 +
> > .../radiusclient-ng/radiusclient-ng_0.5.6.bb | 36 ++++++++++
> > 3 files changed, 116 insertions(+)
> > create mode 100644 meta-networking/recipes-support/radiusclient-ng/files/Fix-configure-and-compile-errors.patch
> > create mode 100644 meta-networking/recipes-support/radiusclient-ng/files/config-site.radiusclient-ng-0.5.6
> > create mode 100644 meta-networking/recipes-support/radiusclient-ng/radiusclient-ng_0.5.6.bb
>
> Can you move this to recipes-protocols?
and files -> radiusclient-ng for slightly faster lookup
Also small nitpick in couple recipes from fujitsu.com I've noticed that you're adding
space before ':' in subject, it should be
<recipe>: some message
I'm changing it when cherry-picking to master-next, but please do it on
your end so that we have less work integrating the patches.
>
> > diff --git a/meta-networking/recipes-support/radiusclient-ng/files/Fix-configure-and-compile-errors.patch b/meta-networking/recipes-support/radiusclient-ng/files/Fix-configure-and-compile-errors.patch
> > new file mode 100644
> > index 0000000..5220923
> > --- /dev/null
> > +++ b/meta-networking/recipes-support/radiusclient-ng/files/Fix-configure-and-compile-errors.patch
> > @@ -0,0 +1,79 @@
> > +From 72154b3179d9352d3747d7b47d6029a9f1a304b2 Mon Sep 17 00:00:00 2001
> > +From: Li xin <lixin.fnst at cn.fujitsu.com>
> > +Date: Tue, 2 Dec 2014 02:33:53 +0900
> > +Subject: [PATCH] Fix configure and compile errors
> > +
> > +Signed-off-by: Li Xin <lixin.fnst at cn.fujitsu.com>
>
> We'll need an upstream-status on this, even if you're just integrating
> Fedora patches. If you are the author of this patch, could you
> elaborate a bit in your comments here? Looking at the Makefile.am patch
> I think I understand the intent, but it's probably best to write it out.
>
> > +---
> > + configure.in | 6 +++++-
> > + etc/Makefile.am | 9 ++++++---
> > + etc/radiusclient.conf.in | 2 +-
> > + 3 files changed, 12 insertions(+), 5 deletions(-)
> > +
> > +diff --git a/configure.in b/configure.in
> > +index 812f640..3a8af99 100644
> > +--- a/configure.in
> > ++++ b/configure.in
> > +@@ -29,6 +29,9 @@ AC_SUBST(LIBVERSION)
> > + pkgsysconfdir=${sysconfdir}/$PACKAGE
> > + AC_SUBST(pkgsysconfdir)
> > +
> > ++pkgdatadir=${datadir}/$PACKAGE
> > ++AC_SUBST(pkgdatadir)
> > ++
> > + AC_PROG_LIBTOOL
> > +
> > + AC_PROG_CC
> > +@@ -118,7 +121,8 @@ then
> > + ],
> > + AC_MSG_RESULT(yes)
> > + AC_DEFINE(HAVE_STRUCT_UTSNAME_DOMAINNAME),
> > +- AC_MSG_RESULT(no)
> > ++ AC_MSG_RESULT(no),
> > ++ AC_MSG_RESULT(assume no),
> > + )
> > + fi
> > +
> > +diff --git a/etc/Makefile.am b/etc/Makefile.am
> > +index 39b6975..a3c403f 100644
> > +--- a/etc/Makefile.am
> > ++++ b/etc/Makefile.am
> > +@@ -14,9 +14,11 @@ CLEANFILES = *~ radiusclient.conf
> > +
> > + sbindir = @sbindir@
> > + pkgsysconfdir = @pkgsysconfdir@
> > +-pkgsysconf_DATA = issue port-id-map radiusclient.conf \
> > +- dictionary dictionary.ascend dictionary.compat dictionary.merit \
> > +- dictionary.sip
> > ++pkgsysconf_DATA = issue port-id-map radiusclient.conf
> > ++
> > ++pkgdatadir = @pkgdatadir@
> > ++pkgdata_DATA = dictionary dictionary.ascend dictionary.compat \
> > ++ dictionary.merit dictionary.sip
> > +
> > + EXTRA_DIST = issue port-id-map dictionary dictionary.ascend \
> > + dictionary.compat dictionary.merit servers radiusclient.conf.in \
> > +@@ -25,6 +27,7 @@ EXTRA_DIST = issue port-id-map dictionary dictionary.ascend \
> > + radiusclient.conf: radiusclient.conf.in
> > + sed -e 's|@sbin''dir@|$(sbindir)|g' \
> > + -e 's|@pkgsysconf''dir@|$(pkgsysconfdir)|g' \
> > ++ -e 's|@pkgdata''dir@|$(pkgdatadir)|g' \
> > + <$(srcdir)/radiusclient.conf.in >radiusclient.conf
> > +
> > + install-data-local: servers
> > +diff --git a/etc/radiusclient.conf.in b/etc/radiusclient.conf.in
> > +index 948bc2f..aac9973 100644
> > +--- a/etc/radiusclient.conf.in
> > ++++ b/etc/radiusclient.conf.in
> > +@@ -47,7 +47,7 @@ servers @pkgsysconfdir@/servers
> > +
> > + # dictionary of allowed attributes and values
> > + # just like in the normal RADIUS distributions
> > +-dictionary @pkgsysconfdir@/dictionary
> > ++dictionary @pkgdatadir@/dictionary
> > +
> > + # program to call for a RADIUS authenticated login
> > + login_radius @sbindir@/login.radius
> > +--
> > +1.8.4.2
> > +
> > diff --git a/meta-networking/recipes-support/radiusclient-ng/files/config-site.radiusclient-ng-0.5.6 b/meta-networking/recipes-support/radiusclient-ng/files/config-site.radiusclient-ng-0.5.6
> > new file mode 100644
> > index 0000000..a30c144
> > --- /dev/null
> > +++ b/meta-networking/recipes-support/radiusclient-ng/files/config-site.radiusclient-ng-0.5.6
> > @@ -0,0 +1 @@
> > +ac_cv_func_uname=no
> > diff --git a/meta-networking/recipes-support/radiusclient-ng/radiusclient-ng_0.5.6.bb b/meta-networking/recipes-support/radiusclient-ng/radiusclient-ng_0.5.6.bb
> > new file mode 100644
> > index 0000000..bc9d08e
> > --- /dev/null
> > +++ b/meta-networking/recipes-support/radiusclient-ng/radiusclient-ng_0.5.6.bb
> > @@ -0,0 +1,36 @@
> > +SUMMARY = "RADIUS protocol client library"
> > +DESCRIPTION = "Portable, easy-to-use and standard compliant library suitable \
> > +for developing free and commercial software that need support for a RADIUS \
> > +protocol (RFCs 2128 and 2139)."
> > +HOMEPAGE = "http://developer.berlios.de/projects/radiusclient-ng/"
>
> This 404s for me. Actually, developer.berlios.de NXDOMAINs. :-)
>
> I was ready to write this off as abandonware since I know a bit of the
> history of freeradius, but it actually looks like it's still relatively
> actively developed on SF. So can you update the HOMEPAGE to point to
> http://sourceforge.net/projects/radiusclient-ng.berlios/ (or some more
> appropriate page, if you've got one).
>
> > +SECTION = "Applications/Internet"
> > +
> > +SRC_URI = "http://pkgs.fedoraproject.org/repo/pkgs/${PN}/${BP}.tar.gz/6fb7d4d0aefafaee7385831ac46a8e9c/${BP}.tar.gz \
> > + file://Fix-configure-and-compile-errors.patch \
> > + file://config-site.radiusclient-ng-${PV}"
> > +
> > +SRC_URI[md5sum] = "6fb7d4d0aefafaee7385831ac46a8e9c"
> > +SRC_URI[sha256sum] = "282a9f1355f190efbb06c0d7c4e062eaa652caf342ed3ad361ac595f72f09f14"
> > +
> > +LICENSE = "BSD"
> > +LIC_FILES_CHKSUM = "file://COPYRIGHT;md5=3e47566c9271b786693d8a08792dbf41"
> > +
> > +inherit autotools-brokensep
> > +
> > +EXTRA_OECONF += "--disable-static"
> > +
> > +do_configure_prepend () {
> > + export CONFIG_SITE=./config-site.${P}
> > +}
> > +
> > +do_compile_prepend() {
> > + for m in `find . -name "Makefile"` ; do
> > + sed -i -e 's:^program_transform_name =.*:program_transform_name =:g' ${m}
> > + done
> > +}
> > +
> > +do_install() {
> > + oe_runmake DESTDIR=${D} install
> > + rm -f ${D}${libdir}/*.la
> > + rm -f ${D}${sbindir}/radexample
> > +}
> > --
> > 1.8.4.2
> >
> --
> -Joe MacDonald.
> :wq
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
--
Martin 'JaMa' Jansa jabber: Martin.Jansa at gmail.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://lists.openembedded.org/pipermail/openembedded-devel/attachments/20141203/85cf1d82/attachment-0002.sig>
More information about the Openembedded-devel
mailing list