Re: [[meta-networking][PATCH v2] tinyproxy: Add recipe

[Benjamin Gaignard <benjamin.gaignard@linaro.org>]

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



-- 
Benjamin Gaignard

Graphic Study Group

Linaro.org │ Open source software for ARM SoCs

Follow Linaro: Facebook | Twitter | Blog