emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
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

  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).