Package Details: astroid 0.15-4

Git Clone URL: (read-only, click to copy)
Package Base: astroid
Description: a graphical threads-with-tags style, lightweight and fast, email client for notmuch, inspired by sup and others
Upstream URL:
Keywords: mail
Licenses: GPL
Submitter: gauteh
Maintainer: gauteh (jheinz)
Last Packager: jheinz
Votes: 26
Popularity: 0.192880
First Submitted: 2015-09-20 10:24
Last Updated: 2019-07-19 11:14

Latest Comments

1 2 3 4 5 6 ... Next › Last »

mathieu.clabaut commented on 2021-06-18 09:23

Note that rebuilding from master branch works.

mathieu.clabaut commented on 2021-06-18 09:04

Same problem here. Reported upstream

psamim commented on 2021-06-12 19:48

I get the following error when trying to install:

../src/ error: ‘_1’ was not declared in this scope
  263 |         bind (&MainWindow::jump_to_page, this, _1, 1));
      |                                                ^~
../src/ note: suggested alternatives:
In file included from /usr/include/boost/mpl/aux_/include_preprocessed.hpp:37,
                 from /usr/include/boost/mpl/placeholders.hpp:43,
                 from /usr/include/boost/iterator/iterator_categories.hpp:16,
                 from /usr/include/boost/iterator/iterator_facade.hpp:13,
                 from /usr/include/boost/filesystem/path.hpp:29,
                 from /usr/include/boost/filesystem.hpp:16,
                 from ../src/
/usr/include/boost/mpl/aux_/preprocessed/gcc/placeholders.hpp:29:16: note:   ‘mpl_::_1’
   29 | typedef arg<1> _1;
      |                ^~
In file included from /usr/include/sigc++-2.0/sigc++/reference_wrapper.h:22,
                 from /usr/include/sigc++-2.0/sigc++/adaptors/bound_argument.h:24,
                 from /usr/include/sigc++-2.0/sigc++/adaptors/bind.h:6,
                 from /usr/include/sigc++-2.0/sigc++/adaptors/adaptors.h:23,
                 from /usr/include/sigc++-2.0/sigc++/sigc++.h:126,
                 from /usr/include/glibmm-2.4/glibmm/thread.h:50,
                 from /usr/include/glibmm-2.4/glibmm.h:103,
                 from /usr/include/gtkmm-3.0/gtkmm.h:100,
                 from ../src/
/usr/include/c++/11.1.0/functional:225:34: note:   ‘std::placeholders::_1’
  225 |     extern const _Placeholder<1> _1;
      |                                  ^~
In file included from /usr/include/boost/mpl/aux_/include_preprocessed.hpp:37,
                 from /usr/include/boost/mpl/placeholders.hpp:43,
                 from /usr/include/boost/iterator/iterator_categories.hpp:16,
                 from /usr/include/boost/iterator/iterator_facade.hpp:13,
                 from /usr/include/boost/filesystem/path.hpp:29,
                 from /usr/include/boost/filesystem.hpp:16,
                 from ../src/
/usr/include/boost/mpl/aux_/preprocessed/gcc/placeholders.hpp:29:16: note:   ‘mpl_::_1’
   29 | typedef arg<1> _1;
      |                ^~
In file included from /usr/include/boost/bind/bind.hpp:2336,
                 from /usr/include/boost/multi_index/sequenced_index.hpp:17,
                 from /usr/include/boost/property_tree/ptree.hpp:23,
                 from ../src/astroid.hh:8,
                 from ../src/
/usr/include/boost/bind/placeholders.hpp:58:38: note:   ‘boost::placeholders::_1’
   58 | BOOST_STATIC_CONSTEXPR boost::arg<1> _1;
      |                                      ^~
In file included from /usr/include/boost/parameter/aux_/is_placeholder.hpp:46,
                 from /usr/include/boost/parameter/value_type.hpp:101,
                 from /usr/include/boost/parameter/aux_/arg_list.hpp:1207,
                 from /usr/include/boost/parameter/aux_/tagged_argument.hpp:76,
                 from /usr/include/boost/parameter/aux_/tag.hpp:11,
                 from /usr/include/boost/parameter/keyword.hpp:10,
                 from /usr/include/boost/log/keywords/severity.hpp:18,
                 from /usr/include/boost/log/trivial.hpp:22,
                 from ../src/astroid.hh:10,
                 from ../src/
/usr/include/boost/mp11/bind.hpp:45:7: note:   ‘boost::mp11::_1’
   45 | using _1 = mp_arg<0>;
      |       ^~
In file included from /usr/include/boost/mpl/aux_/include_preprocessed.hpp:37,
                 from /usr/include/boost/mpl/placeholders.hpp:43,
                 from /usr/include/boost/iterator/iterator_categories.hpp:16,
                 from /usr/include/boost/iterator/iterator_facade.hpp:13,
                 from /usr/include/boost/filesystem/path.hpp:29,
                 from /usr/include/boost/filesystem.hpp:16,
                 from ../src/
/usr/include/boost/mpl/aux_/preprocessed/gcc/placeholders.hpp:29:16: note:   ‘mpl_::_1’
   29 | typedef arg<1> _1;
      |                ^~
[10/110] Building CXX object CMakeFiles/hypocycloid.dir/src/
../src/ In member function ‘void Astroid::ComposeMessage::build()’:
../src/ warning: ‘contentStream’ may be used uninitialized in this function [-Wmaybe-uninitialized]
  241 |         GMimeDataWrapper * contentWrapper = g_mime_data_wrapper_new_with_stream(contentStream, GMIME_CONTENT_ENCODING_DEFAULT);
      |                                             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
[15/110] Building CXX object CMakeFiles/hypocycloid.dir/src/
ninja: build stopped: subcommand failed.

Celti commented on 2020-07-02 21:08

do not symlink, you need to rebuild astroid against the new version of protobuf.

ohli commented on 2020-07-02 20:58

Fails to open, complains about missing Maybe an ln -s to the right one might help?

Celti commented on 2019-11-26 09:06

You have a fair point regarding the license. Okay, install all three of, COPYING.GPL-3.0+, and COPYING.LGPL-2.1+ to /usr/share/licenses/${pkgname}/ and set the licenses array to custom.

As for the transitive dependencies, I specified the two that I caught: gmime3 and gobject-introspection-runtime. There are mixed opinions on whether they should or should not be added; namcap will call them unnecessary.

jheinz commented on 2019-11-26 08:55

Hi Celti,

thanks a lot for your input and the patch. I'll update the package today after running some tests on my own.

Regarding the license: I think it's a bit more complicated: The GMime part is licensed under LGPL v2.1 or later

Regarding dependencies: Thanks a lot for cleaning things up. I'll need to check if there's no transitive dependency in your definitions, e.g. will astroid break if notmuch drops its gmime3 dependency (for whatever reason)? If so, we still need to name it.

Cheers, Johannes

Celti commented on 2019-11-26 02:13

The install script is unnecessary; the gtk-update-icon-cache pacman hook has been doing that job for years. Along with that, any empty fields (e.g., epoch, groups, provides, conflicts, replaces, backup, options, changelog, andnoextract`) should be removed to improve clarity.

Astroid is specifically licensed under GPL v3 or any later version; the PKGBUILD should read LICENSE=('GPL3') to reflect this.

There are multiple depends=() issues: * notmuch-runtime instead of notmuch
* boost is a make dependency, boost-libs remains as the runtime dependency.
* gmime3 is a dependency of notmuch and may be removed here.
* gobject-introspection is a make dependency. The runtime dependency, gobject-introspection-runtime, is a dependency of libpeas and does not need to be added here.

There are multiple makedepends=() issues: * As above, boost and gobject-introspection should be added here.
* git is not necessary here.
* pkg-config should not be listed here; its provider, pkgconf, is part of base-devel.
* python-gobject is not necessary here.
* vte3 is listed in optional dependencies, but should be added here as well to ensure it is available to build against.

The check() function fails to run entirely when building in a clean chroot (e.g., with makechrootpkg) due to the inability to access a display server; fix this by adding xorg-server-xvfb to checkdepends and calling xvfb-run -a ninja test instead of ctest --output-on-failure. Additionally, as notmuch-runtime is a full dependency it should be dropped from checkdepends=().

A minimal patch for all of the above (tested) changes follows:

diff --git a/PKGBUILD b/PKGBUILD
index 909cc7e..63ccc33 100644
@@ -4,2 +4 @@ pkgver=0.15
@@ -9,5 +8,4 @@ url=""
-depends=('notmuch' 'boost' 'boost-libs' 'gmime3' 'gtkmm3' 'webkit2gtk' 'libsass' 'libpeas' 'gobject-introspection' 'protobuf')
-makedepends=('cmake' 'ninja' 'git' 'pkg-config' 'python-gobject' 'cmark' 'ruby-ronn' 'w3m')
+depends=('notmuch-runtime' 'boost-libs' 'gtkmm3' 'webkit2gtk' 'libsass' 'libpeas' 'protobuf')
+makedepends=('cmake' 'ninja' 'boost' 'gobject-introspection' 'cmark' 'ruby-ronn' 'vte3' 'w3m')
@@ -18,7 +15,0 @@ optdepends=('gvim: default editor'
@@ -27 +17,0 @@ source=("${pkgname}-v${pkgver}.tar.gz::
@@ -47 +37 @@ check() {
-  ctest --output-on-failure
+  xvfb-run -a ninja test

gauteh commented on 2019-07-18 17:43

Great, that should make life easier. Added you as co-maintainer.

jheinz commented on 2019-07-18 13:57

@gauteh sure, I can give it a try :)

Also, I noticed that notmuch seems to be built against gmime3 now: