diff mbox

[oe,[meta-networking,v2] tinyproxy: Add recipe

Message ID 1491486145-30397-1-git-send-email-benjamin.gaignard@linaro.org
State Superseded
Headers show

Commit Message

Benjamin Gaignard April 6, 2017, 1:42 p.m. UTC
Signed-off-by: Benjamin Gaignard <benjamin.gaignard@linaro.org>

---
version 2:
- create user and group for tinyproxy
- only install the service if systemd is present in DISTRO_FEATURE
- add dependency on asciidoc-native
 .../tinyproxy/tinyproxy/tinyproxy.service          | 14 ++++++++
 .../recipes-support/tinyproxy/tinyproxy_1.8.4.bb   | 41 ++++++++++++++++++++++
 2 files changed, 55 insertions(+)
 create mode 100644 meta-networking/recipes-support/tinyproxy/tinyproxy/tinyproxy.service
 create mode 100644 meta-networking/recipes-support/tinyproxy/tinyproxy_1.8.4.bb

-- 
1.9.1

-- 
_______________________________________________
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel

Comments

Martin Jansa April 12, 2017, 12:45 p.m. UTC | #1
Still fails to build:
http://errors.yoctoproject.org/Errors/Details/140323/

On Thu, Apr 6, 2017 at 3:42 PM, Benjamin Gaignard <
benjamin.gaignard@linaro.org> wrote:

> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@linaro.org>

> ---

> version 2:

> - create user and group for tinyproxy

> - only install the service if systemd is present in DISTRO_FEATURE

> - add dependency on asciidoc-native

>  .../tinyproxy/tinyproxy/tinyproxy.service          | 14 ++++++++

>  .../recipes-support/tinyproxy/tinyproxy_1.8.4.bb   | 41

> ++++++++++++++++++++++

>  2 files changed, 55 insertions(+)

>  create mode 100644 meta-networking/recipes-support/tinyproxy/tinyproxy/

> tinyproxy.service

>  create mode 100644 meta-networking/recipes-support/tinyproxy/tinyproxy_1.

> 8.4.bb

>

> diff --git a/meta-networking/recipes-support/tinyproxy/tinyproxy/tinyproxy.service

> b/meta-networking/recipes-support/tinyproxy/tinyproxy/tinyproxy.service

> new file mode 100644

> index 0000000..ea079f9

> --- /dev/null

> +++ b/meta-networking/recipes-support/tinyproxy/tinyproxy/

> tinyproxy.service

> @@ -0,0 +1,14 @@

> +[Unit]

> +Description=Tinyproxy, Lightweight http(s) proxy daemon

> +RequiresMountsFor=/var

> +

> +[Service]

> +Group=tinyproxy

> +ExecStartPre=/bin/mkdir -p /var/log/tinyproxy

> +ExecStartPre=/bin/chown -R nobody.tinyproxy /var/log/tinyproxy

> +ExecStartPre=/bin/mkdir -p /var/run/tinyproxy

> +ExecStartPre=/bin/chown -R nobody.tinyproxy /var/run/tinyproxy

> +ExecStart=/usr/sbin/tinyproxy -d

> +

> +[Install]

> +WantedBy=multi-user.target

> diff --git a/meta-networking/recipes-support/tinyproxy/tinyproxy_1.8.4.bb

> b/meta-networking/recipes-support/tinyproxy/tinyproxy_1.8.4.bb

> new file mode 100644

> index 0000000..ef2a2d8

> --- /dev/null

> +++ b/meta-networking/recipes-support/tinyproxy/tinyproxy_1.8.4.bb

> @@ -0,0 +1,41 @@

> +SUMMARY = "Lightweight http(s) proxy daemon"

> +HOMEPAGE = "https://tinyproxy.github.io/"

> +LICENSE = "GPLv2"

> +LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe"

> +

> +SRC_URI = "https://github.com/${BPN}/${BPN}/releases/download/${PV}/$

> {BP}.tar.gz \

> +                  file://tinyproxy.service"

> +

> +SRC_URI[md5sum] = "3b60f7d08e0821ed1a3e2cf1e5778cac"

> +SRC_URI[sha256sum] = "8234c879a129feee61efa98bac14a1

> a3e46e5cf08f01696a216940872aa70faf"

> +

> +DEPENDS = "asciidoc-native"

> +

> +EXTRA_OECONF += " \

> +       --enable-filter \

> +       --enable-transparent \

> +       --disable-regexcheck \

> +       --enable-reverse \

> +       --enable-upstream \

> +       --enable-xtinyproxy \

> +       "

> +

> +inherit autotools systemd useradd

> +

> +#User specific

> +USERADD_PACKAGES = "${PN}"

> +USERADD_PARAM_${PN} = "nobody"

> +GROUPADD_PARAM_${PN} = "--system tinyproxy"

> +

> +SYSTEMD_PACKAGES += "${BPN}"

> +SYSTEMD_SERVICE_${PN} = "tinyproxy.service"

> +SYSTEMD_AUTO_ENABLE_${PN} = "enable"

> +

> +do_install_append() {

> +       if ${@bb.utils.contains('DISTRO_FEATURES', 'systemd', 'true',

> 'false', d)}; then

> +               install -d ${D}${systemd_system_unitdir}

> +               install -m 0644 ${WORKDIR}/tinyproxy.service

> ${D}${systemd_system_unitdir}

> +       fi

> +}

> +

> +FILES_${PN} =+ "${systemd_system_unitdir}/tinyproxy.service"

> --

> 1.9.1

>

>

-- 
_______________________________________________
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel
Benjamin Gaignard April 12, 2017, 6:46 p.m. UTC | #2
2017-04-12 14:45 GMT+02:00 Martin Jansa <martin.jansa@gmail.com>:
> Still fails to build:

> http://errors.yoctoproject.org/Errors/Details/140323/


That is an error I never had in my setup.
I guess that libxml2 native is missing but I have way to reproduce the problem.
How is it possible to perform the same build then you ?

>

> On Thu, Apr 6, 2017 at 3:42 PM, Benjamin Gaignard

> <benjamin.gaignard@linaro.org> wrote:

>>

>> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@linaro.org>

>> ---

>> version 2:

>> - create user and group for tinyproxy

>> - only install the service if systemd is present in DISTRO_FEATURE

>> - add dependency on asciidoc-native

>>  .../tinyproxy/tinyproxy/tinyproxy.service          | 14 ++++++++

>>  .../recipes-support/tinyproxy/tinyproxy_1.8.4.bb   | 41

>> ++++++++++++++++++++++

>>  2 files changed, 55 insertions(+)

>>  create mode 100644

>> meta-networking/recipes-support/tinyproxy/tinyproxy/tinyproxy.service

>>  create mode 100644

>> meta-networking/recipes-support/tinyproxy/tinyproxy_1.8.4.bb

>>

>> diff --git

>> a/meta-networking/recipes-support/tinyproxy/tinyproxy/tinyproxy.service

>> b/meta-networking/recipes-support/tinyproxy/tinyproxy/tinyproxy.service

>> new file mode 100644

>> index 0000000..ea079f9

>> --- /dev/null

>> +++

>> b/meta-networking/recipes-support/tinyproxy/tinyproxy/tinyproxy.service

>> @@ -0,0 +1,14 @@

>> +[Unit]

>> +Description=Tinyproxy, Lightweight http(s) proxy daemon

>> +RequiresMountsFor=/var

>> +

>> +[Service]

>> +Group=tinyproxy

>> +ExecStartPre=/bin/mkdir -p /var/log/tinyproxy

>> +ExecStartPre=/bin/chown -R nobody.tinyproxy /var/log/tinyproxy

>> +ExecStartPre=/bin/mkdir -p /var/run/tinyproxy

>> +ExecStartPre=/bin/chown -R nobody.tinyproxy /var/run/tinyproxy

>> +ExecStart=/usr/sbin/tinyproxy -d

>> +

>> +[Install]

>> +WantedBy=multi-user.target

>> diff --git a/meta-networking/recipes-support/tinyproxy/tinyproxy_1.8.4.bb

>> b/meta-networking/recipes-support/tinyproxy/tinyproxy_1.8.4.bb

>> new file mode 100644

>> index 0000000..ef2a2d8

>> --- /dev/null

>> +++ b/meta-networking/recipes-support/tinyproxy/tinyproxy_1.8.4.bb

>> @@ -0,0 +1,41 @@

>> +SUMMARY = "Lightweight http(s) proxy daemon"

>> +HOMEPAGE = "https://tinyproxy.github.io/"

>> +LICENSE = "GPLv2"

>> +LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe"

>> +

>> +SRC_URI =

>> "https://github.com/${BPN}/${BPN}/releases/download/${PV}/${BP}.tar.gz \

>> +                  file://tinyproxy.service"

>> +

>> +SRC_URI[md5sum] = "3b60f7d08e0821ed1a3e2cf1e5778cac"

>> +SRC_URI[sha256sum] =

>> "8234c879a129feee61efa98bac14a1a3e46e5cf08f01696a216940872aa70faf"

>> +

>> +DEPENDS = "asciidoc-native"

>> +

>> +EXTRA_OECONF += " \

>> +       --enable-filter \

>> +       --enable-transparent \

>> +       --disable-regexcheck \

>> +       --enable-reverse \

>> +       --enable-upstream \

>> +       --enable-xtinyproxy \

>> +       "

>> +

>> +inherit autotools systemd useradd

>> +

>> +#User specific

>> +USERADD_PACKAGES = "${PN}"

>> +USERADD_PARAM_${PN} = "nobody"

>> +GROUPADD_PARAM_${PN} = "--system tinyproxy"

>> +

>> +SYSTEMD_PACKAGES += "${BPN}"

>> +SYSTEMD_SERVICE_${PN} = "tinyproxy.service"

>> +SYSTEMD_AUTO_ENABLE_${PN} = "enable"

>> +

>> +do_install_append() {

>> +       if ${@bb.utils.contains('DISTRO_FEATURES', 'systemd', 'true',

>> 'false', d)}; then

>> +               install -d ${D}${systemd_system_unitdir}

>> +               install -m 0644 ${WORKDIR}/tinyproxy.service

>> ${D}${systemd_system_unitdir}

>> +       fi

>> +}

>> +

>> +FILES_${PN} =+ "${systemd_system_unitdir}/tinyproxy.service"

>> --

>> 1.9.1

>>

>

-- 
_______________________________________________
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel
Martin Jansa April 12, 2017, 7:04 p.m. UTC | #3
If you're testing it with master, then the xmllint from host cannot be used
because of HOSTPATH filtering recently merged.

For other configuration you can see:
http://www.openembedded.org/wiki/Bitbake_World_Status_Setup


On Wed, Apr 12, 2017 at 8:46 PM, Benjamin Gaignard <
benjamin.gaignard@linaro.org> wrote:

> 2017-04-12 14:45 GMT+02:00 Martin Jansa <martin.jansa@gmail.com>:

> > Still fails to build:

> > http://errors.yoctoproject.org/Errors/Details/140323/

>

> That is an error I never had in my setup.

> I guess that libxml2 native is missing but I have way to reproduce the

> problem.

> How is it possible to perform the same build then you ?

>

> >

> > On Thu, Apr 6, 2017 at 3:42 PM, Benjamin Gaignard

> > <benjamin.gaignard@linaro.org> wrote:

> >>

> >> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@linaro.org>

> >> ---

> >> version 2:

> >> - create user and group for tinyproxy

> >> - only install the service if systemd is present in DISTRO_FEATURE

> >> - add dependency on asciidoc-native

> >>  .../tinyproxy/tinyproxy/tinyproxy.service          | 14 ++++++++

> >>  .../recipes-support/tinyproxy/tinyproxy_1.8.4.bb   | 41

> >> ++++++++++++++++++++++

> >>  2 files changed, 55 insertions(+)

> >>  create mode 100644

> >> meta-networking/recipes-support/tinyproxy/tinyproxy/tinyproxy.service

> >>  create mode 100644

> >> meta-networking/recipes-support/tinyproxy/tinyproxy_1.8.4.bb

> >>

> >> diff --git

> >> a/meta-networking/recipes-support/tinyproxy/tinyproxy/tinyproxy.service

> >> b/meta-networking/recipes-support/tinyproxy/tinyproxy/tinyproxy.service

> >> new file mode 100644

> >> index 0000000..ea079f9

> >> --- /dev/null

> >> +++

> >> b/meta-networking/recipes-support/tinyproxy/tinyproxy/tinyproxy.service

> >> @@ -0,0 +1,14 @@

> >> +[Unit]

> >> +Description=Tinyproxy, Lightweight http(s) proxy daemon

> >> +RequiresMountsFor=/var

> >> +

> >> +[Service]

> >> +Group=tinyproxy

> >> +ExecStartPre=/bin/mkdir -p /var/log/tinyproxy

> >> +ExecStartPre=/bin/chown -R nobody.tinyproxy /var/log/tinyproxy

> >> +ExecStartPre=/bin/mkdir -p /var/run/tinyproxy

> >> +ExecStartPre=/bin/chown -R nobody.tinyproxy /var/run/tinyproxy

> >> +ExecStart=/usr/sbin/tinyproxy -d

> >> +

> >> +[Install]

> >> +WantedBy=multi-user.target

> >> diff --git a/meta-networking/recipes-support/tinyproxy/tinyproxy_1.

> 8.4.bb

> >> b/meta-networking/recipes-support/tinyproxy/tinyproxy_1.8.4.bb

> >> new file mode 100644

> >> index 0000000..ef2a2d8

> >> --- /dev/null

> >> +++ b/meta-networking/recipes-support/tinyproxy/tinyproxy_1.8.4.bb

> >> @@ -0,0 +1,41 @@

> >> +SUMMARY = "Lightweight http(s) proxy daemon"

> >> +HOMEPAGE = "https://tinyproxy.github.io/"

> >> +LICENSE = "GPLv2"

> >> +LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882b

> be"

> >> +

> >> +SRC_URI =

> >> "https://github.com/${BPN}/${BPN}/releases/download/${PV}/${BP}.tar.gz

> \

> >> +                  file://tinyproxy.service"

> >> +

> >> +SRC_URI[md5sum] = "3b60f7d08e0821ed1a3e2cf1e5778cac"

> >> +SRC_URI[sha256sum] =

> >> "8234c879a129feee61efa98bac14a1a3e46e5cf08f01696a216940872aa70faf"

> >> +

> >> +DEPENDS = "asciidoc-native"

> >> +

> >> +EXTRA_OECONF += " \

> >> +       --enable-filter \

> >> +       --enable-transparent \

> >> +       --disable-regexcheck \

> >> +       --enable-reverse \

> >> +       --enable-upstream \

> >> +       --enable-xtinyproxy \

> >> +       "

> >> +

> >> +inherit autotools systemd useradd

> >> +

> >> +#User specific

> >> +USERADD_PACKAGES = "${PN}"

> >> +USERADD_PARAM_${PN} = "nobody"

> >> +GROUPADD_PARAM_${PN} = "--system tinyproxy"

> >> +

> >> +SYSTEMD_PACKAGES += "${BPN}"

> >> +SYSTEMD_SERVICE_${PN} = "tinyproxy.service"

> >> +SYSTEMD_AUTO_ENABLE_${PN} = "enable"

> >> +

> >> +do_install_append() {

> >> +       if ${@bb.utils.contains('DISTRO_FEATURES', 'systemd', 'true',

> >> 'false', d)}; then

> >> +               install -d ${D}${systemd_system_unitdir}

> >> +               install -m 0644 ${WORKDIR}/tinyproxy.service

> >> ${D}${systemd_system_unitdir}

> >> +       fi

> >> +}

> >> +

> >> +FILES_${PN} =+ "${systemd_system_unitdir}/tinyproxy.service"

> >> --

> >> 1.9.1

> >>

> >

>

-- 
_______________________________________________
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel
Andre McCurdy April 12, 2017, 7:53 p.m. UTC | #4
On Wed, Apr 12, 2017 at 12:04 PM, Martin Jansa <martin.jansa@gmail.com> wrote:
> If you're testing it with master, then the xmllint from host cannot be used

> because of HOSTPATH filtering recently merged.

>

> For other configuration you can see:

> http://www.openembedded.org/wiki/Bitbake_World_Status_Setup

>

> On Wed, Apr 12, 2017 at 8:46 PM, Benjamin Gaignard <

> benjamin.gaignard@linaro.org> wrote:

>

>> 2017-04-12 14:45 GMT+02:00 Martin Jansa <martin.jansa@gmail.com>:

>> > Still fails to build:

>> > http://errors.yoctoproject.org/Errors/Details/140323/

>>

>> That is an error I never had in my setup.

>> I guess that libxml2 native is missing but I have way to reproduce the

>> problem.

>> How is it possible to perform the same build then you ?


You need to make the configure tests for xsltproc, asciidoc and
xmllint deterministic. e.g. to remove dependencies on all three, add:

  ac_cv_path_XSLTPROC=no
  ac_cv_path_A2X=no
  ac_cv_path_XMLLINT=no

to EXTRA_OECONF (which would allow you to remove asciidoc-native from DEPENDS).

>> >

>> > On Thu, Apr 6, 2017 at 3:42 PM, Benjamin Gaignard

>> > <benjamin.gaignard@linaro.org> wrote:

>> >>

>> >> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@linaro.org>

>> >> ---

>> >> version 2:

>> >> - create user and group for tinyproxy

>> >> - only install the service if systemd is present in DISTRO_FEATURE

>> >> - add dependency on asciidoc-native

>> >>  .../tinyproxy/tinyproxy/tinyproxy.service          | 14 ++++++++

>> >>  .../recipes-support/tinyproxy/tinyproxy_1.8.4.bb   | 41

>> >> ++++++++++++++++++++++

>> >>  2 files changed, 55 insertions(+)

>> >>  create mode 100644

>> >> meta-networking/recipes-support/tinyproxy/tinyproxy/tinyproxy.service

>> >>  create mode 100644

>> >> meta-networking/recipes-support/tinyproxy/tinyproxy_1.8.4.bb

>> >>

>> >> diff --git

>> >> a/meta-networking/recipes-support/tinyproxy/tinyproxy/tinyproxy.service

>> >> b/meta-networking/recipes-support/tinyproxy/tinyproxy/tinyproxy.service

>> >> new file mode 100644

>> >> index 0000000..ea079f9

>> >> --- /dev/null

>> >> +++

>> >> b/meta-networking/recipes-support/tinyproxy/tinyproxy/tinyproxy.service

>> >> @@ -0,0 +1,14 @@

>> >> +[Unit]

>> >> +Description=Tinyproxy, Lightweight http(s) proxy daemon

>> >> +RequiresMountsFor=/var

>> >> +

>> >> +[Service]

>> >> +Group=tinyproxy

>> >> +ExecStartPre=/bin/mkdir -p /var/log/tinyproxy

>> >> +ExecStartPre=/bin/chown -R nobody.tinyproxy /var/log/tinyproxy

>> >> +ExecStartPre=/bin/mkdir -p /var/run/tinyproxy

>> >> +ExecStartPre=/bin/chown -R nobody.tinyproxy /var/run/tinyproxy

>> >> +ExecStart=/usr/sbin/tinyproxy -d

>> >> +

>> >> +[Install]

>> >> +WantedBy=multi-user.target

>> >> diff --git a/meta-networking/recipes-support/tinyproxy/tinyproxy_1.

>> 8.4.bb

>> >> b/meta-networking/recipes-support/tinyproxy/tinyproxy_1.8.4.bb

>> >> new file mode 100644

>> >> index 0000000..ef2a2d8

>> >> --- /dev/null

>> >> +++ b/meta-networking/recipes-support/tinyproxy/tinyproxy_1.8.4.bb

>> >> @@ -0,0 +1,41 @@

>> >> +SUMMARY = "Lightweight http(s) proxy daemon"

>> >> +HOMEPAGE = "https://tinyproxy.github.io/"

>> >> +LICENSE = "GPLv2"

>> >> +LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882b

>> be"

>> >> +

>> >> +SRC_URI =

>> >> "https://github.com/${BPN}/${BPN}/releases/download/${PV}/${BP}.tar.gz

>> \

>> >> +                  file://tinyproxy.service"

>> >> +

>> >> +SRC_URI[md5sum] = "3b60f7d08e0821ed1a3e2cf1e5778cac"

>> >> +SRC_URI[sha256sum] =

>> >> "8234c879a129feee61efa98bac14a1a3e46e5cf08f01696a216940872aa70faf"

>> >> +

>> >> +DEPENDS = "asciidoc-native"

>> >> +

>> >> +EXTRA_OECONF += " \

>> >> +       --enable-filter \

>> >> +       --enable-transparent \

>> >> +       --disable-regexcheck \

>> >> +       --enable-reverse \

>> >> +       --enable-upstream \

>> >> +       --enable-xtinyproxy \

>> >> +       "

>> >> +

>> >> +inherit autotools systemd useradd

>> >> +

>> >> +#User specific

>> >> +USERADD_PACKAGES = "${PN}"

>> >> +USERADD_PARAM_${PN} = "nobody"

>> >> +GROUPADD_PARAM_${PN} = "--system tinyproxy"

>> >> +

>> >> +SYSTEMD_PACKAGES += "${BPN}"

>> >> +SYSTEMD_SERVICE_${PN} = "tinyproxy.service"

>> >> +SYSTEMD_AUTO_ENABLE_${PN} = "enable"

>> >> +

>> >> +do_install_append() {

>> >> +       if ${@bb.utils.contains('DISTRO_FEATURES', 'systemd', 'true',

>> >> 'false', d)}; then

>> >> +               install -d ${D}${systemd_system_unitdir}

>> >> +               install -m 0644 ${WORKDIR}/tinyproxy.service

>> >> ${D}${systemd_system_unitdir}

>> >> +       fi

>> >> +}

>> >> +

>> >> +FILES_${PN} =+ "${systemd_system_unitdir}/tinyproxy.service"


This shouldn't be required if SYSTEMD_SERVICE_${PN} includes
"tinyproxy.service".

>> >> --

>> >> 1.9.1

>> >>

>> >

>>

> --

> _______________________________________________

> Openembedded-devel mailing list

> Openembedded-devel@lists.openembedded.org

> http://lists.openembedded.org/mailman/listinfo/openembedded-devel

-- 
_______________________________________________
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel
Benjamin Gaignard April 13, 2017, 12:31 p.m. UTC | #5
2017-04-12 21:53 GMT+02:00 Andre McCurdy <armccurdy@gmail.com>:
> On Wed, Apr 12, 2017 at 12:04 PM, Martin Jansa <martin.jansa@gmail.com> wrote:
>> If you're testing it with master, then the xmllint from host cannot be used
>> because of HOSTPATH filtering recently merged.
>>
>> For other configuration you can see:
>> http://www.openembedded.org/wiki/Bitbake_World_Status_Setup
>>
>> On Wed, Apr 12, 2017 at 8:46 PM, Benjamin Gaignard <
>> benjamin.gaignard@linaro.org> wrote:
>>
>>> 2017-04-12 14:45 GMT+02:00 Martin Jansa <martin.jansa@gmail.com>:
>>> > Still fails to build:
>>> > http://errors.yoctoproject.org/Errors/Details/140323/
>>>
>>> That is an error I never had in my setup.
>>> I guess that libxml2 native is missing but I have way to reproduce the
>>> problem.
>>> How is it possible to perform the same build then you ?
>
> You need to make the configure tests for xsltproc, asciidoc and
> xmllint deterministic. e.g. to remove dependencies on all three, add:
>
>   ac_cv_path_XSLTPROC=no
>   ac_cv_path_A2X=no
>   ac_cv_path_XMLLINT=no

I don't know what that could break in tinyproxy so I prefer to add the
dependencies.

>
> to EXTRA_OECONF (which would allow you to remove asciidoc-native from DEPENDS).
>
>>> >
>>> > On Thu, Apr 6, 2017 at 3:42 PM, Benjamin Gaignard
>>> > <benjamin.gaignard@linaro.org> wrote:
>>> >>
>>> >> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@linaro.org>
>>> >> ---
>>> >> version 2:
>>> >> - create user and group for tinyproxy
>>> >> - only install the service if systemd is present in DISTRO_FEATURE
>>> >> - add dependency on asciidoc-native
>>> >>  .../tinyproxy/tinyproxy/tinyproxy.service          | 14 ++++++++
>>> >>  .../recipes-support/tinyproxy/tinyproxy_1.8.4.bb   | 41
>>> >> ++++++++++++++++++++++
>>> >>  2 files changed, 55 insertions(+)
>>> >>  create mode 100644
>>> >> meta-networking/recipes-support/tinyproxy/tinyproxy/tinyproxy.service
>>> >>  create mode 100644
>>> >> meta-networking/recipes-support/tinyproxy/tinyproxy_1.8.4.bb
>>> >>
>>> >> diff --git
>>> >> a/meta-networking/recipes-support/tinyproxy/tinyproxy/tinyproxy.service
>>> >> b/meta-networking/recipes-support/tinyproxy/tinyproxy/tinyproxy.service
>>> >> new file mode 100644
>>> >> index 0000000..ea079f9
>>> >> --- /dev/null
>>> >> +++
>>> >> b/meta-networking/recipes-support/tinyproxy/tinyproxy/tinyproxy.service
>>> >> @@ -0,0 +1,14 @@
>>> >> +[Unit]
>>> >> +Description=Tinyproxy, Lightweight http(s) proxy daemon
>>> >> +RequiresMountsFor=/var
>>> >> +
>>> >> +[Service]
>>> >> +Group=tinyproxy
>>> >> +ExecStartPre=/bin/mkdir -p /var/log/tinyproxy
>>> >> +ExecStartPre=/bin/chown -R nobody.tinyproxy /var/log/tinyproxy
>>> >> +ExecStartPre=/bin/mkdir -p /var/run/tinyproxy
>>> >> +ExecStartPre=/bin/chown -R nobody.tinyproxy /var/run/tinyproxy
>>> >> +ExecStart=/usr/sbin/tinyproxy -d
>>> >> +
>>> >> +[Install]
>>> >> +WantedBy=multi-user.target
>>> >> diff --git a/meta-networking/recipes-support/tinyproxy/tinyproxy_1.
>>> 8.4.bb
>>> >> b/meta-networking/recipes-support/tinyproxy/tinyproxy_1.8.4.bb
>>> >> new file mode 100644
>>> >> index 0000000..ef2a2d8
>>> >> --- /dev/null
>>> >> +++ b/meta-networking/recipes-support/tinyproxy/tinyproxy_1.8.4.bb
>>> >> @@ -0,0 +1,41 @@
>>> >> +SUMMARY = "Lightweight http(s) proxy daemon"
>>> >> +HOMEPAGE = "https://tinyproxy.github.io/"
>>> >> +LICENSE = "GPLv2"
>>> >> +LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882b
>>> be"
>>> >> +
>>> >> +SRC_URI =
>>> >> "https://github.com/${BPN}/${BPN}/releases/download/${PV}/${BP}.tar.gz
>>> \
>>> >> +                  file://tinyproxy.service"
>>> >> +
>>> >> +SRC_URI[md5sum] = "3b60f7d08e0821ed1a3e2cf1e5778cac"
>>> >> +SRC_URI[sha256sum] =
>>> >> "8234c879a129feee61efa98bac14a1a3e46e5cf08f01696a216940872aa70faf"
>>> >> +
>>> >> +DEPENDS = "asciidoc-native"
>>> >> +
>>> >> +EXTRA_OECONF += " \
>>> >> +       --enable-filter \
>>> >> +       --enable-transparent \
>>> >> +       --disable-regexcheck \
>>> >> +       --enable-reverse \
>>> >> +       --enable-upstream \
>>> >> +       --enable-xtinyproxy \
>>> >> +       "
>>> >> +
>>> >> +inherit autotools systemd useradd
>>> >> +
>>> >> +#User specific
>>> >> +USERADD_PACKAGES = "${PN}"
>>> >> +USERADD_PARAM_${PN} = "nobody"
>>> >> +GROUPADD_PARAM_${PN} = "--system tinyproxy"
>>> >> +
>>> >> +SYSTEMD_PACKAGES += "${BPN}"
>>> >> +SYSTEMD_SERVICE_${PN} = "tinyproxy.service"
>>> >> +SYSTEMD_AUTO_ENABLE_${PN} = "enable"
>>> >> +
>>> >> +do_install_append() {
>>> >> +       if ${@bb.utils.contains('DISTRO_FEATURES', 'systemd', 'true',
>>> >> 'false', d)}; then
>>> >> +               install -d ${D}${systemd_system_unitdir}
>>> >> +               install -m 0644 ${WORKDIR}/tinyproxy.service
>>> >> ${D}${systemd_system_unitdir}
>>> >> +       fi
>>> >> +}
>>> >> +
>>> >> +FILES_${PN} =+ "${systemd_system_unitdir}/tinyproxy.service"
>
> This shouldn't be required if SYSTEMD_SERVICE_${PN} includes
> "tinyproxy.service".

Yes I will remove it.

>>> >> --
>>> >> 1.9.1
>>> >>
>>> >
>>>
>> --
>> _______________________________________________
>> Openembedded-devel mailing list
>> Openembedded-devel@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
diff mbox

Patch

diff --git a/meta-networking/recipes-support/tinyproxy/tinyproxy/tinyproxy.service b/meta-networking/recipes-support/tinyproxy/tinyproxy/tinyproxy.service
new file mode 100644
index 0000000..ea079f9
--- /dev/null
+++ b/meta-networking/recipes-support/tinyproxy/tinyproxy/tinyproxy.service
@@ -0,0 +1,14 @@ 
+[Unit]
+Description=Tinyproxy, Lightweight http(s) proxy daemon
+RequiresMountsFor=/var
+
+[Service]
+Group=tinyproxy
+ExecStartPre=/bin/mkdir -p /var/log/tinyproxy
+ExecStartPre=/bin/chown -R nobody.tinyproxy /var/log/tinyproxy
+ExecStartPre=/bin/mkdir -p /var/run/tinyproxy
+ExecStartPre=/bin/chown -R nobody.tinyproxy /var/run/tinyproxy
+ExecStart=/usr/sbin/tinyproxy -d
+
+[Install]
+WantedBy=multi-user.target
diff --git a/meta-networking/recipes-support/tinyproxy/tinyproxy_1.8.4.bb b/meta-networking/recipes-support/tinyproxy/tinyproxy_1.8.4.bb
new file mode 100644
index 0000000..ef2a2d8
--- /dev/null
+++ b/meta-networking/recipes-support/tinyproxy/tinyproxy_1.8.4.bb
@@ -0,0 +1,41 @@ 
+SUMMARY = "Lightweight http(s) proxy daemon"
+HOMEPAGE = "https://tinyproxy.github.io/"
+LICENSE = "GPLv2"
+LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe"
+
+SRC_URI = "https://github.com/${BPN}/${BPN}/releases/download/${PV}/${BP}.tar.gz \
+		   file://tinyproxy.service"
+
+SRC_URI[md5sum] = "3b60f7d08e0821ed1a3e2cf1e5778cac"
+SRC_URI[sha256sum] = "8234c879a129feee61efa98bac14a1a3e46e5cf08f01696a216940872aa70faf"
+
+DEPENDS = "asciidoc-native"
+
+EXTRA_OECONF += " \
+	--enable-filter \
+	--enable-transparent \
+	--disable-regexcheck \
+	--enable-reverse \
+	--enable-upstream \
+	--enable-xtinyproxy \
+	"
+
+inherit autotools systemd useradd
+
+#User specific
+USERADD_PACKAGES = "${PN}"
+USERADD_PARAM_${PN} = "nobody"
+GROUPADD_PARAM_${PN} = "--system tinyproxy"
+
+SYSTEMD_PACKAGES += "${BPN}"
+SYSTEMD_SERVICE_${PN} = "tinyproxy.service"
+SYSTEMD_AUTO_ENABLE_${PN} = "enable"
+
+do_install_append() {
+	if ${@bb.utils.contains('DISTRO_FEATURES', 'systemd', 'true', 'false', d)}; then
+		install -d ${D}${systemd_system_unitdir}
+		install -m 0644 ${WORKDIR}/tinyproxy.service ${D}${systemd_system_unitdir}
+	fi
+}
+
+FILES_${PN} =+ "${systemd_system_unitdir}/tinyproxy.service"