[Top][All Lists]

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

Re: [O] Changes to contrib

From: Rasmus
Subject: Re: [O] Changes to contrib
Date: Mon, 09 Nov 2015 22:16:17 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.0.50 (gnu/linux)


Kosyrev Serge <address@hidden> writes:

> Rasmus <address@hidden> writes:
>> Serge Kosyrev <address@hidden> writes:
>>> I'm not sure how wise it would be to raise barriers for contribution,
>>> given the current state of the thing..
>> I don't know what state you refer to.
> Well, I couldn't use it, without significant tweaking.
> It feels as if the thing doesn't have a maintainer and slowly decomposes.

The question is whether there's any desire down the road to move this
backend to core?  Given that I don’t know that paper status of two former
contributors, and nobody chimed in, it’s likely that that race is already

> The question is.. the only documentation for ox-taskjuggler that exists
> in Worg, seems to only cover the surface aspects of export -- none of
> the existing documentation touches on the multitude of details that the
> existing code does involve itself with.

OK.  There should be docstring and potentially comments, if necessary.
Worg is a separate issue.

>>>> I think you add some more taskjuggle keywords/properties for
>>>> example.
> I do, indeed -- and they are undocumented in a matter that is similar to
> the 80% of pre-existing properties.

OK.  It would be better to have some sort of documentation there, but
that’s a separate bug.

>> You use string-join, which is in subr-x.  I think subr-x was not a
>> dependency before.
> Oh, indeed -- missed that!
> What should I do about it?

I’d just use mapconcat TBH, since it’s just one place as I recall.

    (equal (string-join '("a" "b") " ")
           (mapconcat 'identity '("a" "b") " "))

> Understood, will rename, then.

Thanks.  And thanks for working on this!


This message is brought to you by the department of redundant departments

reply via email to

[Prev in Thread] Current Thread [Next in Thread]