pkgsrc-Changes archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: pkgsrc/textproc/groonga



On Tue, 25 Feb 2014 12:06:38 +0900, Joerg Sonnenberger 
<joerg%britannica.bec.de@localhost> wrote:

On Tue, Feb 25, 2014 at 11:13:05AM +0900, OBATA Akio wrote:
On Tue, 25 Feb 2014 10:01:17 +0900, Joerg Sonnenberger 
<joerg%britannica.bec.de@localhost> wrote:

>On Tue, Feb 25, 2014 at 12:58:30AM +0000, OBATA Akio wrote:
>>Module Name:    pkgsrc
>>Committed By:   obache
>>Date:           Tue Feb 25 00:58:30 UTC 2014
>>
>>Modified Files:
>>        pkgsrc/textproc/groonga: Makefile
>>Removed Files:
>>        pkgsrc/textproc/groonga/patches: patch-lib_Makefile.in
>>
>>Log Message:
>>Marked as broken with clang (it should be fixed instead).
>
>Please revert it, this is rude.

If you want to revert, please take the maintainership of this package
and care future update, if you really want to use this package with clang.
You may grad to fix with such one-shot-patch, but it will make it
hard for me to update for future upstream release.

Given that you haven't really cared in the past either, it doesn't
change anything...

Could you try to disable optimization globally with BUILDLINK_TRANSFORM
in hacks.mk?

That's significantly worse and a bad excuse when it is trivial to apply
the optimisation in the one place that caes.

It is the usually manner used in various other packages.
I have not heard the issue on other platforms using clang (I'll confirm
it from upstream), so it may be restricted more closely to your environment.

It is a well understood problem with this type of source code. It will
run out of memory or become very large, depending on the available
resources.

I'm talking about following...
* reduce total maintenance cost
* not happened on other than your environment

Again, are you really using Groonga?
Are you glad to drop optimization for such parts?

Anyway, your bulk build result should be happy now.

--
OBATA Akio / obata%lins.jp@localhost



Home | Main Index | Thread Index | Old Index