[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/7] gnu: Add mate-polkit.
From: |
Andreas Enge |
Subject: |
Re: [PATCH 1/7] gnu: Add mate-polkit. |
Date: |
Sun, 21 Feb 2016 16:26:19 +0100 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
Hello,
On Sat, Feb 20, 2016 at 06:17:28PM +0100, Fabian Harfert wrote:
> I carefully checked all the library source dependencies and so on. I
> think all these propagated inputs are needed.
okay, excellent.
> If that is necessary, I'll add these comments, but I think it's not too
> difficult to look into the .pc files. Some of the packages have
> comments, but the most don't.
Well, it would be useful in the future, as some kind of proof that you had
a look at it, and also to keep track if people wonder why it has been
done like this. If there is a change in the future and something is not
needed any more, this would then become clear simply by looking at the
referenced .pc file.
> But actually I don't really know. I'll recheck all the .pc files and
> headers (not too much).
I think there is no need to check the headers; the example I gave was
already treated correctly. If a header is needed, most of the time the
library will also be mentioned in the .pc file.
I will let you have a look at the glib-or-gtk-build-system question,
and give us notice again when the next patches are ready. On the other
hand, if there are mistakes, that would not be a big problem; when you
notice in the end that something does not work, it will still be time
to add (propagated or other) inputs.
Thanks for all this work!
Andreas
- [PATCH 1/7] gnu: Add mate-polkit., Fabian Harfert, 2016/02/19
- [PATCH 5/7] gnu: Add mate-session-manager., Fabian Harfert, 2016/02/19
- [PATCH 3/7] gnu: Add libmatemixer., Fabian Harfert, 2016/02/19
- [PATCH 2/7] gnu: Add libmatekbd., Fabian Harfert, 2016/02/19
- [PATCH 4/7] gnu: Add mate-settings-daemon., Fabian Harfert, 2016/02/19
- [PATCH 6/7] gnu: Add marco., Fabian Harfert, 2016/02/19
- [PATCH 7/7] gnu: Add mate-panel., Fabian Harfert, 2016/02/19
- Re: [PATCH 1/7] gnu: Add mate-polkit., Andreas Enge, 2016/02/20