From: Takashi Sakamoto <o-takashi@sakamocchi.jp>
To: Takashi Iwai <tiwai@suse.de>
Cc: alsa-devel@alsa-project.org, clemens@ladisch.de
Subject: Re: [PATCH 00/25] alsa-tools: efw-downloader: add initial version of firmwre downloader for Echo Audio Fireworks devices
Date: Fri, 28 Aug 2020 01:02:12 +0900 [thread overview]
Message-ID: <20200827160212.GA406268@workstation> (raw)
In-Reply-To: <s5hd03cp7q7.wl-tiwai@suse.de>
Hi,
On Thu, Aug 27, 2020 at 02:57:04PM +0200, Takashi Iwai wrote:
> On Thu, 27 Aug 2020 14:24:42 +0200, Takashi Sakamoto wrote:
> > On Wed, Aug 26, 2020 at 01:31:53PM +0200, Takashi Iwai wrote:
> > > Since it's based on meson build, it'll be tricky to include this in
> > > alsa-tools whether others are all autoconf. The tarball creation is
> > > done in the top directory and that assumes the execution of "make
> > > alsa-dist" in each subdirectory. Without this integration, the
> > > directory won't be included in the release.
> > >
> > > Could you work on it, too?
> >
> > I didn't have enough care of distributing the package. Thank you for the
> > indication.
> >
> > Although it's possible to write configure.ac/Makefile.am for
> > efw-downloader, I'd like to use meson.build for my convenience, especially
> > for the convenience of gnome module[1] in meson (Nowadays software in GNOME
> > project including GLib is mostly build by meson).
> >
> > As long as I know, the concept of release creation in GNU Autotools is
> > different from the one in meson build system. GNU Autotools distributes
> > scripts generated from Makefile.am/configure.ac and so. On the other
> > hand, meson distributes files maintained by git or mercurial.
> >
> > If we have a space to make enough arrangement for alsa-tools, the
> > top-level Makefile should be changed to have two variables for
> > subdirectories which includes software built by GNU Autotools and the
> > others, then be changed further for configure/install/alsa-dist/clean
> > targets.
> >
> > Nevertheless, the idea to mix all of software built by several types of
> > build system into one repository is not so convenient itself. I'll take
> > more time to investigate further for better packaging of alsa-tools.
> > (Tools like Android repo is a bit over-engineering in the case, mmm)
> >
> > I decline the patchset for now.
>
> OK. It's indeed awkward to mix up both auto-tools and meson.
> So the more suitable option would be either to modernize everything
> with meson, or just create a configure.ac for efw-downloader.
> Maybe the latter is easier, as the dependency would be only about
> hinawa and the check via pkgconfig is trivial even with automake.
> But, obviously, modernization is more appealing (with a risk of
> breakage, as always :)
The modernization is itself preferrable, but the idea of everything with
meson is not better since we have several build systems in the world. In
my opinion, the preferrable way is to enable developers to add software
without limitations about its build system and dependency.
As a quick glance, below applications have dependency to Gtk+2 or Gtk+3.
For them, replacement with meson build system is reasonable:
* echomixer
* envy24control
* hdajackretask
* rmedigicontrol
As you know, Gtk+2 is already obsoleted, thus the above should be ported to
Gtk4[1].
Qlo10k1 is only an application of Qt3. I guess CMake is more preferrable
than GNU Autotools in the case. As well as Gtk+2, Gt3 is already
obsoleted.
Hwmixvolume is written by Python 3, thus it's better to follow the
standard way in Python world (setup.py).
For the other software, it doesn't matter still to use GNU autotools.
However, ld10k1 includes tools (ld10k1/lo10k1/dl10k1) seem to depend on
local library (liblo10k1) but Makefile seems not to describe the dependency
appropriately.
At present, I have no proposal for the issue, but it's possible to
split the software into several repositories depending on build system,
like:
* alsa-tools-meson
* alsa-tools-cmake
* alsa-tools-python
* alsa-tools-autotools
Then put release script to alsa-tools repository with git-submodules for them.
[1] Debian Bug report logs - #967250
alsa-tools: depends on deprecated GTK 2
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=967250
Regards
Takashi Sakamoto
prev parent reply other threads:[~2020-08-27 16:03 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-08-21 7:30 [PATCH 00/25] alsa-tools: efw-downloader: add initial version of firmwre downloader for Echo Audio Fireworks devices Takashi Sakamoto
2020-08-21 7:30 ` [PATCH 01/25] efw-downloader: start a new project to operate on-board flash memory for Fireworks board module Takashi Sakamoto
2020-08-21 7:30 ` [PATCH 02/25] efw-downloader: efw-proto: define EfwProto as derived object of HinawaFwResp Takashi Sakamoto
2020-08-21 7:30 ` [PATCH 03/25] efw-downloader: efw-proto: add constructor, destructor, bind, unbind functions Takashi Sakamoto
2020-08-21 7:30 ` [PATCH 04/25] efw-downloader: efw-proto: add responded signal Takashi Sakamoto
2020-08-21 7:30 ` [PATCH 05/25] efw-downloader: efw-proto: add class virtual method to handle " Takashi Sakamoto
2020-08-21 7:30 ` [PATCH 06/25] efw-downloader: efw-proto: add instance private structure Takashi Sakamoto
2020-08-21 7:30 ` [PATCH 07/25] efw-downloader: efw-proto: emit responded signal at receiving response Takashi Sakamoto
2020-08-21 7:30 ` [PATCH 08/25] efw-downloader: efw-proto: add function to finish transaction for command frame Takashi Sakamoto
2020-08-21 7:30 ` [PATCH 09/25] efw-downloader: efw-proto: add function to finish a pair of transactions Takashi Sakamoto
2020-08-21 7:30 ` [PATCH 10/25] efw-downloader: add parser for sub commands Takashi Sakamoto
2020-08-21 7:30 ` [PATCH 11/25] efw-downloader: subcmd-device: implement 'device' sub command to operate actual device Takashi Sakamoto
2020-08-21 7:30 ` [PATCH 12/25] efw-downloader: subcmd-device: open firewire character device by HinawaFwNode Takashi Sakamoto
2020-08-21 7:30 ` [PATCH 13/25] efw-downloader: config-rom: parse config rom to detect supported device Takashi Sakamoto
2020-08-21 7:31 ` [PATCH 14/25] efw-downloader: subcmd-device: check supported models or not Takashi Sakamoto
2020-08-21 7:31 ` [PATCH 15/25] efw-downloader: subcmd-device: bind Fireworks protocol Takashi Sakamoto
2020-08-21 7:31 ` [PATCH 16/25] efw-downloader: node_dispatcher: add event dispatcher utilizing GLib MainContext/MainLoop Takashi Sakamoto
2020-08-21 7:31 ` [PATCH 17/25] efw-downloader: subcmd-device: support debug output for response of Fireworks protocol Takashi Sakamoto
2020-08-21 7:31 ` [PATCH 18/25] efw-downloader: efw-commands: add support for a part of hardware command Takashi Sakamoto
2020-08-21 7:31 ` [PATCH 19/25] efw-downloader: efw-commands: add support for commands in flash category Takashi Sakamoto
2020-08-21 7:31 ` [PATCH 20/25] efw-downloader: subcmd-device: add read operation Takashi Sakamoto
2020-08-21 7:31 ` [PATCH 21/25] efw-downloader: file-cntr: add parser for data binary shipped by Echo Digital Audio corporation Takashi Sakamoto
2020-08-21 7:31 ` [PATCH 22/25] efw-downloader: subcmd-file: add 'file' sub command Takashi Sakamoto
2020-08-21 7:31 ` [PATCH 23/25] efw-downloader: subcmd-file: add parse operation Takashi Sakamoto
2020-08-21 7:31 ` [PATCH 24/25] efw-downloader: man: add online manual Takashi Sakamoto
2020-08-21 7:31 ` [PATCH 25/25] efw-downloader: add README formatted by reStructuredText Takashi Sakamoto
2020-08-26 9:32 ` [PATCH 00/25] alsa-tools: efw-downloader: add initial version of firmwre downloader for Echo Audio Fireworks devices Takashi Sakamoto
2020-08-26 11:31 ` Takashi Iwai
2020-08-27 12:24 ` Takashi Sakamoto
2020-08-27 12:57 ` Takashi Iwai
2020-08-27 16:02 ` Takashi Sakamoto [this message]
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=20200827160212.GA406268@workstation \
--to=o-takashi@sakamocchi.jp \
--cc=alsa-devel@alsa-project.org \
--cc=clemens@ladisch.de \
--cc=tiwai@suse.de \
/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.