From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kosyrev Serge <_deepfire@feelingofgreen.ru> Subject: Re: Changes to contrib Date: Tue, 10 Nov 2015 00:05:59 +0300 Message-ID: <87fv0enars.fsf@feelingofgreen.ru> References: <87vb9cdauy.fsf@andromedae.feelingofgreen.ru> <87bnb4y527.fsf@gmx.us> <87mvumnbrp.fsf@feelingofgreen.ru> <87si4erj1c.fsf@gmx.us> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:49569) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from <_deepfire@feelingofgreen.ru>) id 1Zvtdd-0007yn-Oh for emacs-orgmode@gnu.org; Mon, 09 Nov 2015 16:06:06 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from <_deepfire@feelingofgreen.ru>) id 1ZvtdZ-0003QD-8U for emacs-orgmode@gnu.org; Mon, 09 Nov 2015 16:06:05 -0500 Received: from [80.92.100.69] (port=45574 helo=mail.feelingofgreen.ru) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from <_deepfire@feelingofgreen.ru>) id 1ZvtdZ-0003Q3-26 for emacs-orgmode@gnu.org; Mon, 09 Nov 2015 16:06:01 -0500 In-Reply-To: <87si4erj1c.fsf@gmx.us> (sfid-20151110_001704_217768_F0712ACD) (rasmus@gmx.us's message of "Mon, 09 Nov 2015 21:53:51 +0100") List-Id: "General discussions about Org-mode." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org Sender: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org To: Rasmus Cc: emacs-orgmode@gnu.org Rasmus writes: > Serge Kosyrev <_deepfire@feelingofgreen.ru> 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. >>> Some quick comments from skimming your code (note, I have no idea what a >>> taskjuggler is): >>> >>> I don=E2=80=99t know what you refer to explicitly. But that should be = fixed, I >>> guess. >> >> I'm sorry, what should be? > > Sorry, I was referring to this quote by you, which seems to have > disappeared: > >>>> (One immediate nitpick, of course, is that none of the additions >>>> are documented..) That is true, yes. 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. The point I'm trying to make, is that producing some documentation that would cover the details (that one actually expects to be covered, coming from the TaskJuggler background) seems like a separate task. That is, not in scope of my little "tweak it to work for myself" series.. >>> 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. >> [...] >>> Also, you introduce a dependency on subr-x, which may or may not be an >>> issue since it=E2=80=99s in contrib. >> >> I'm sorry, I'm not sure what do you mean? I looked at subr-x.el, and >> I didn't find any reference to any of those functions.. > > 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? >>> org-export-map-special-nodes and org-element-multivalued-property are n= ot >>> proper names, as they will supposedly only be used in ox-taskjuggler. >> >> Well, the functions themselves are supposed to be generic. >> >> There are two ways to name a function, in my mind: >> >> - by intended application >> - by what it does >> >> ..it's just that I chose the second.. in the futile hope that someone, >> sometime will move/use them. : -) > > Then they are in the wrong library. > >> If you deem it not to be an appropriate course of thought, I will >> change the name, sure. > > At this point, they lack appropriate names. Understood, will rename, then. --=20 =D1=81 =D1=83=D0=B2=D0=B0=D0=B6=D0=B5=D0=BD=D0=B8e=D0=BC / respectfully, =D0=9A=D0=BE=D1=81=D1=8B=D1=80=D0=B5=D0=B2 =D0=A1=D0=B5=D1=80=D0=B3=D0=B5= =D0=B9