From: Carsten Dominik <carsten.dominik@gmail.com>
To: "Tom Breton (Tehom)" <tehom@panix.com>
Cc: emacs-orgmode@gnu.org
Subject: Re: org-html link building diff
Date: Sun, 16 May 2010 07:03:22 +0200 [thread overview]
Message-ID: <9D0A8669-60C2-462F-8235-881573A664D6@gmail.com> (raw)
In-Reply-To: <b0abdc48bcf2c73f733a45ad390e4b5c.squirrel@mail.panix.com>
Hi Tom,
I have now taken a brief look at the html-export-refactor-build-link
patch. I see that in addition to changing org-html, it create a new
file, with tests. While it is great that you have defined tests, this
meant that I cannot simply apply he patch to the master without making
unwanted changes.
If we build a test strucure, it should be in a completely separate
directory, not in, for example, the lisp directory.
Could you please make me a branch which contains only the changes that
would go into org?
And if you want to provide a test framework for Org, pleas, by all
means, go ahead, build it and document it so that we can all use it.
Maybe we could make a "tests" subdirectory in the git repo that would
contain it...
Thanks
- Carsten
On May 15, 2010, at 11:37 PM, Tom Breton (Tehom) wrote:
>> Hi Tom,
>>
>> what is the status of this development? Ready for prime time? If
>> yes, can you please rebase to master and send me the pointer to the
>> branch again?
>
> I believe it is ready, though I haven't heard any feedback. I
> pushed the
> changes on about the 4th of may.
>
>> Or is there still stuff unclear? Can I help?
>
> Maybe. I hadn't done this before, so some feedback and (if needed)
> correction would be appreciated.
>
> I had to change the url to git+ssh so it would authenticate me, but
> when I
> pushed, it was visible in the public (non-ssh) git repo, so I
> thought it
> was visible to everybody. Isn't it?
>
> I thought I had done what you wanted by creating a branch for all my
> changes ever (tehom-master) and rebasing the org-html link changes
> on that
> (html-export-refactor-build-link). Is that working for you? If
> not, how
> can I fix it?
>
> Tom Breton (Tehom)
>
>
- Carsten
next prev parent reply other threads:[~2010-05-16 5:03 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-04-29 22:24 org-html link building diff Tom Breton (Tehom)
2010-05-01 12:01 ` Carsten Dominik
2010-05-15 12:29 ` Carsten Dominik
2010-05-15 21:37 ` Tom Breton (Tehom)
2010-05-16 5:03 ` Carsten Dominik [this message]
2010-05-18 0:59 ` Tom Breton (Tehom)
2010-05-18 4:47 ` Carsten Dominik
2010-05-18 12:26 ` Sebastian Rose
2010-05-16 5:20 ` Carsten Dominik
2010-05-18 1:01 ` Test-file naming conventions - mine and suggested for org Tom Breton (Tehom)
2010-05-21 11:27 ` Carsten Dominik
2010-05-24 21:26 ` Proposal: Emtest as tester Tom Breton (Tehom)
2010-05-24 22:56 ` Dan Davison
2010-05-27 20:02 ` Tom Breton (Tehom)
2010-05-25 6:48 ` Carsten Dominik
2010-05-25 8:43 ` Martin Pohlack
2010-05-27 20:13 ` Tom Breton (Tehom)
2010-06-07 16:11 ` Benjamin Andresen
-- strict thread matches above, loose matches on Subject: below --
2010-04-18 2:13 org-html link building diff Tom Breton (Tehom)
2010-04-26 5:24 ` Carsten Dominik
2010-04-26 10:03 ` Sebastian Rose
2010-04-26 19:45 ` Tom Breton (Tehom)
2010-04-27 6:07 ` Carsten Dominik
2010-04-28 3:01 ` Tom Breton (Tehom)
2010-04-28 15:07 ` Carsten Dominik
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.orgmode.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=9D0A8669-60C2-462F-8235-881573A664D6@gmail.com \
--to=carsten.dominik@gmail.com \
--cc=emacs-orgmode@gnu.org \
--cc=tehom@panix.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).