Re: [O] [PATCH] Possible bug in org-protocol

Hello Nicolas, Nicolas Goaziou writes: > ... >> Is simple diffs an acceptable format for patches or should I send you an >> fully formatted git commit ? > > The latter, please. > > Could you send a complete version of the patch? What dou you think of the attached one ? Regards, Pierre >From

Re: [O] [PATCH] Possible bug in org-protocol

Hello Nicolas, Nicolas Goaziou writes: > Hello, > > pierre.techouey...@free.fr (Pierre Téchoueyres) writes: > >> I think so, as org-protocol, for what I've understood, is triggered by >> an URL where protocol is org-protocol. So the content of the URL itself

Re: [O] [PATCH] Possible bug in org-protocol

Hi Nicolas, Nicolas Goaziou writes: >> >> 2) use emacsclient to trigger org-protocol like this >> >> #+begin_src sh >> emacsclient -n >> "org-protocol://capture?template=p&url=https%3A%2F%2Fgithub.com%2FCruiserX%2Fsha256_plsql&title=CruiserX%2Fsha256_plsql%3A%20SHA256%20PL%2FSQL%20Implementat

[O] [PATCH] Possible bug in org-protocol

Hello Org developers, I'm facing an unexpected behaviour since some days : When I want to record some link from web sites using Org Capture extension (https://github.com/sprig/org-capture-extension) for Firefox all my text and links are encoded like urls. It's look like the commit #cc7c4a673 rem

Re: [O] [PATCH] Add new keyword :coding for #+include directive

Hello nicolas, Many thanks for your patience and for applying my patch. I've seen you didn't apply the part which added an entry in ORG-NEWS file. Was this deliberate or just an side effect of your refactoring ? Or is this unneeded ? Again thanks for your patience. Nicolas Goaziou writes: > .

Re: [O] [PATCH] Add new keyword :coding for #+include directive

l variables for example). But sometimes, there is no such information and you can not modify the file (for various reasons. ex: the file is generated automatically). Once again, I'm sorry for the disturbance. Regards, Nicolas Goaziou writes: > Hello, > > pierre.techouey...@free.f

Re: [O] [PATCH] Add new keyword :coding for #+include directive

Org mode version 9.1.13 (release_9.1.13-763-gcd25c7 @ /home/ptechoueyres/Travail/VCS/emacs-org-mode/lisp/) So I don't really understand what's appening. Nicolas Goaziou writes: > Hello, > > pierre.techouey...@free.fr (Pierre Téchoueyres) writes: >

Re: [O] [PATCH] Add new keyword :coding for #+include directive

Hello, And sorry for the delay. Nicolas Goaziou writes: > Hello, > > pierre.techouey...@free.fr (Pierre Téchoueyres) writes: > >> Hello Nicolas, >> Did you have time to review the patches ? > > Sorry for the delay, I have been sidetracked. > > I admit I don&

Re: [O] [PATCH] Add new keyword :coding for #+include directive

Hello Nicolas, Did you have time to review the patches ? pierre.techouey...@free.fr (Pierre Téchoueyres) writes: > Hello, > Nicolas Goaziou writes: > >> Hello, >> >> pierre.techouey...@free.fr (Pierre Téchoueyres) writes: >> >>> I think I've

Re: [O] [PATCH] Add new keyword :coding for #+include directive

Hello, Nicolas Goaziou writes: > Hello, > > pierre.techouey...@free.fr (Pierre Téchoueyres) writes: > >> I think I've corrected all points. You'll find new versions attached. > > Thank you. > >> Would you mind consider to include the patch for the

Re: [O] [PATCH] Add new keyword :coding for #+include directive

Hello Nicolas, I think I've corrected all points. You'll find new versions attached. Would you mind consider to include the patch for the detection of encoding with the #+include keyword in 9.2 release ? >From e4d6cf8f9959781c682738a4b9e5ea8ae5747b6b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pierr

Re: [O] [PATCH] Add new keyword :coding for #+include directive

pierre.techouey...@free.fr (Pierre Téchoueyres) writes: Hello, > ... >> ... >> Is it really an Org problem? E.g., couldn't you put a coding: cookie in >> your ".cmd" file? IMO, the coding system depends on the includee, not >> the includer. > >

Re: [O] [PATCH] Add new keyword :coding for #+include directive

inside the included file. But : a) This seem to not work as expected (see exemples joinned). b) Sometimes you can't modify the included file (ex: remote file access). # -*- coding: utf-8 -*- #+title: test with different encodings. * Pierre Téchoueyres bad #+include: "./file.

Re: [O] [PATCH] Add new keyword :coding for #+include directive

Hello org's developpers, Sorry, I forgot the patch. so here is the whole mail + patch: I want to propose the attached patch which allow you to specify an optionnal `:coding' keyword to the `#+INCLUDE:' directive. This allow you to specify something like #+begin_example ,#+INCLUDE: "myfile.cmd" s

[O] [PATCH] Add new keyword :coding for #+include directive

Hello org's developpers, I want to propose the attached patch which allow you to specify an optionnal `:coding' keyword to the `#+INCLUDE:' directive. This allow you to specify something like #+begin_example ,#+INCLUDE: "myfile.cmd" src cmd :coding "cp850-dos" #+end_example Which allow you to ha

[O] [PATCH] org-agenda-list: call recenter only when buffer is visible

Hello org's developers, I'm trying to correct the package `org-alert' (https://github.com/spegoraro/org-alert) in order to not interfer with the org-agenda buffer. I've tried to call `org-agenda-list' inside a temp buffer. This could work but the call to `recenter' throw an error which defeat the

Re: [O] [PATCH] ob-sql.el: Improve Oracle connection and usage for ob-sql.

Nicolas Goaziou writes: Hi Nicolas, > ... > It looks good. > > I added two spaces at the end of sentences and pushed it. > ... I'm sorry I did a mistake in the previous patch. There is an inconsistency between the code in `org-babel-sql-dbstring-oracle' and the example provided in the ORG-NEWS f

Re: [O] [PATCH] ob-sql.el: Improve Oracle connection and usage for ob-sql.

Nicolas Goaziou writes: > ... > Otherwise, it looks good. Would you mind adding an entry in ORG-NEWS > about it? Hello Nicolas, What dou you think about the attached patch ? >From 0103a07b10915ce7c919b8a6858beff3dbd7e45f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pierre=20T=C3=A9choueyres?= Date:

[O] [PATCH] ob-sql.el: Improve Oracle connection and usage for ob-sql.

Hello all Org's developpers, I would like to provide a small improvement to ob-sql : the possibility to connect to an oracle database by using the alias defined in TNSNAMES file. The patch joined here try to achive this. It also improve (slightly) the data fetching by removing unwanted trailing wh

Re: [O] elpa's copy of org-version.el

see a version like mmdd and no longer see a version with mmdd_hhmm) Hope this could help Pierre Le samedi 20 juillet 2013 16:52:10 Paul Stansell a écrit : > Pierre Téchoueyres free.fr> writes: > > Ok I've found *my* mistake : > > Melpa provides an org-m

Re: [O] elpa's copy of org-version.el

Ok I've found *my* mistake : Melpa provides an org-mode from GIT too. Look like this version is buggy. Pierre Le jeudi 18 juillet 2013 20:41:58 Pierre Téchoueyres a écrit : Same problem here. I've tried the version org-20130718.1557 from elpa too. May be an packaging problem ?

Re: [O] elpa's copy of org-version.el

Same problem here. I've tried the version org-20130718.1557 from elpa too. May be an packaging problem ? Pierre Le jeudi 18 juillet 2013 17:44:06 Paul Stansell a écrit : > Hello, > > If I install org from elpa the file org-version.el is missing and > I get the following: > > M-x org-version >

[Orgmode] [PATCH] Properly export title tag in org-html

Hi List, I think I've found a little bug in org-export-as-html function in org- html.el file. If you're setting an title with some escape character then the title tag in html header is set "as is". for example if your title is foo\_bar then the title tag will be foo\_ba