From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/2] scanpypi: new utility
Date: Wed, 13 Jan 2016 16:23:20 +0100 [thread overview]
Message-ID: <20160113162320.1c1dbbc3@free-electrons.com> (raw)
In-Reply-To: <56927A92.50804@mind.be>
Arnout, Yann,
On Sun, 10 Jan 2016 16:36:50 +0100, Arnout Vandecappelle wrote:
> > So, we currently have scancpan to create perl packages. You are adding
> > scanpypi to create Python packages. There's also someone who submitted
> > a script to generate a 'generic' package (i.e. not perl/python) [0].
> > The scancpan is written in perl, yours and the generic one in Python.
> >
> > Besides Perl and Python, we also have nodejs which provides a similar
> > "package-store" and for which it would become interesting to provide a
> > helper script to generate packages [1].
> >
> > What I would love to see is that we have a single script to add
> > packages. Something like:
> >
> > $ ./support/script/add-package -t TYPE [OPTS] PKG [PKG...]
> >
> > with TYPE being one of the package types we currently have (generic,
> > autotools... python, perl...) or an abstract type (nodejs...).
> >
> > Then, the cpan, pypi, nodejs... script would be just 'backends' that
> > would provide classes called by the main script, like;
> >
> > pkg = new PythonPkg("foo")
> > pkg.get_br_name() returns the BR2_PACKAGE_ name
> > pkg.get_version() returns the _VERSION string
> > pkg.get_source() returns the _SOURCE string
> > pkg.get_site() returns the _SITE string
> > pkg.get_method() returns the _SITE_METHOD string
> > pkg.get_dependencies() returns the _DEPENDENCIES list
> > ... and so on, you get the idea. ;-)
>
> However, it is more natural for a CPAN-accessor to be written in perl. So I
> guess the backend scripts should be really independent scripts that report the
> package metadata in a specified format. Hm, you know what, let's use Config.in
> and pkg.mk as the specification!
>
> In short, I'm not so convinced that having everything written in the same
> language is such an advantage.
>
> But of course, if someone shows me the patches, I could change my mind.
I also initially would have preferred to have the scancpan script
written in Python. But 1/ it's not very practical to query CPAN from
Python, and more importantly 2/ it's a bit weird to ask to a Perl
fan who maintains the Perl stuff in Buildroot to write something like
scancpan in Python.
Bottom line, my opinion is that:
1/ We should keep scancpan in Perl
2/ We should keep scanpipy in Python, refine it and merge it.
3/ The script that generates just a generic package skeleton is a bit
useless IMO and is not very useful to merge.
Both scancpan and canpipy are really internal tools, for Buildroot
developers, so it's not super important if they don't look like /
behave the same.
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
next prev parent reply other threads:[~2016-01-13 15:23 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-28 13:15 [Buildroot] [PATCH 1/2] scanpypi: new utility Denis THULIN
2015-07-28 13:15 ` [Buildroot] [PATCH 2/2] python-robotframework: New package Denis THULIN
2015-08-31 15:58 ` [Buildroot] [PATCH 1/2] scanpypi: new utility Denis Thulin
2016-01-10 10:59 ` Yann E. MORIN
2016-01-10 15:36 ` Arnout Vandecappelle
2016-01-13 15:23 ` Thomas Petazzoni [this message]
2016-01-14 8:32 ` Yegor Yefremov
2016-01-27 13:30 ` Yegor Yefremov
2016-02-02 18:02 ` Eelco Chaudron
2016-02-02 19:54 ` Eelco Chaudron
2016-03-01 1:44 ` Carlos Santos
-- strict thread matches above, loose matches on Subject: below --
2015-07-09 13:31 [Buildroot] [PATCH 0/2] python-package-generator Denis THULIN
2015-07-09 13:31 ` [Buildroot] [PATCH 1/2] scanpypi: new utility Denis THULIN
2015-07-11 12:56 ` Arnout Vandecappelle
2015-07-15 14:08 ` Denis Thulin
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=20160113162320.1c1dbbc3@free-electrons.com \
--to=thomas.petazzoni@free-electrons.com \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox