Re: emacs: Wrong "From" address when forwarding a mail

2015-12-20 Thread Jinwoo Lee
On Sun, Dec 20, 2015 at 12:54 PM, Mark Walters wrote: > You can get around some of this by setting > notmuch-always-prompt-for-sender to t (it is a customizable > variable). This means that whenever you create a new message or forward > a message you will be prompted

emacs: Wrong "From" address when forwarding a mail

2015-12-20 Thread Jinwoo Lee
I have two accounts set up in my notmuch configuration. And when I'm forwarding a mail using notmuch-show-forward-message, it always uses my primary account as its From address no matter which account that original mail has been delivered to. For example: 1. I got a mail delivered to my personal

[PATCH] lib, ruby: make use of -Wl,--no-undefined configurable

2015-06-17 Thread Jinwoo Lee
On Tue, Jun 16, 2015 at 11:23 PM, David Bremner wrote: > At a guess, this again has to do with spaces in the pathname. Maybe this > is fixable with more quoting, but it's really independent of mac vs > gnu/linux. In this instance it looks like quoting "test_results_path" > would help. Ah, I

Re: [PATCH] lib, ruby: make use of -Wl,--no-undefined configurable

2015-06-17 Thread Jinwoo Lee
On Tue, Jun 16, 2015 at 11:23 PM, David Bremner da...@tethera.net wrote: At a guess, this again has to do with spaces in the pathname. Maybe this is fixable with more quoting, but it's really independent of mac vs gnu/linux. In this instance it looks like quoting test_results_path would help.

[PATCH] lib, ruby: make use of -Wl,--no-undefined configurable

2015-06-16 Thread Jinwoo Lee
On Tue, Jun 16, 2015 at 04:43 PM, Jinwoo Lee wrote: > Turns out my previous error was because I was running tests within Emacs > eshell, and run_emacs failed. > > I tried again in a terminal and it still failed but for a different > reason. I don't have readelf on my Mac. L

[PATCH] lib, ruby: make use of -Wl,--no-undefined configurable

2015-06-16 Thread Jinwoo Lee
Turns out my previous error was because I was running tests within Emacs eshell, and run_emacs failed. I tried again in a terminal and it still failed but for a different reason. I don't have readelf on my Mac. $ make test /Applications/Xcode.app/Contents/Developer/usr/bin/make -C .. test cd

Re: [PATCH] lib, ruby: make use of -Wl,--no-undefined configurable

2015-06-16 Thread Jinwoo Lee
On Tue, Jun 16, 2015 at 04:43 PM, Jinwoo Lee jinwo...@gmail.com wrote: Turns out my previous error was because I was running tests within Emacs eshell, and run_emacs failed. I tried again in a terminal and it still failed but for a different reason. I don't have readelf on my Mac. Looks

[PATCH] lib, ruby: make use of -Wl,--no-undefined configurable

2015-06-15 Thread Jinwoo Lee
> That probably will not change; IIRC bash 3.2.57 is released under GPLv2 > and bash >= 4 GPLv3. You just need to install separate bash (brew, ports, > wherever) in order to run these tests in OS X. I installed bash from MacPorts and ran `make test' again, but the result is not so good. The

Re: [PATCH] lib, ruby: make use of -Wl,--no-undefined configurable

2015-06-15 Thread Jinwoo Lee
That probably will not change; IIRC bash 3.2.57 is released under GPLv2 and bash = 4 GPLv3. You just need to install separate bash (brew, ports, wherever) in order to run these tests in OS X. I installed bash from MacPorts and ran `make test' again, but the result is not so good. The error

[PATCH] lib, ruby: make use of -Wl,--no-undefined configurable

2015-06-14 Thread Jinwoo Lee
On Sun, Jun 14, 2015 at 11:34 AM, David Bremner wrote: > Jinwoo Lee writes: > >> When I apply those 2 patches from you, things seem to work. I still get >> warnings like below, but they don't seem severe. I'm not sure if the >> ruby binding actually works though. >

[PATCH] lib, ruby: make use of -Wl,--no-undefined configurable

2015-06-14 Thread Jinwoo Lee
On Sat, Jun 13, 2015 at 11:53 PM, David Bremner wrote: > Jinwoo Lee writes: > >> There are 2 problems. >> >> 1. The file, bindings/Makefile.local still has lib/libnotmuch.so as the >>dependency of ruby-bindings. > > this is fixed in > >

[PATCH] lib, ruby: make use of -Wl,--no-undefined configurable

2015-06-14 Thread Jinwoo Lee
There are 2 problems. 1. The file, bindings/Makefile.local still has lib/libnotmuch.so as the dependency of ruby-bindings. > make: *** No rule to make target `lib/libnotmuch.so', needed by > `ruby-bindings'. Stop. 2. After manually changing it to lib/libnotmuch.dylib, I still get an

Re: [PATCH] lib, ruby: make use of -Wl,--no-undefined configurable

2015-06-14 Thread Jinwoo Lee
There are 2 problems. 1. The file, bindings/Makefile.local still has lib/libnotmuch.so as the dependency of ruby-bindings. make: *** No rule to make target `lib/libnotmuch.so', needed by `ruby-bindings'. Stop. 2. After manually changing it to lib/libnotmuch.dylib, I still get an error

Re: [PATCH] lib, ruby: make use of -Wl,--no-undefined configurable

2015-06-14 Thread Jinwoo Lee
On Sat, Jun 13, 2015 at 11:53 PM, David Bremner da...@tethera.net wrote: Jinwoo Lee jinwo...@gmail.com writes: There are 2 problems. 1. The file, bindings/Makefile.local still has lib/libnotmuch.so as the dependency of ruby-bindings. this is fixed in id:1434263191-14171-1-git

Re: [PATCH] lib, ruby: make use of -Wl,--no-undefined configurable

2015-06-14 Thread Jinwoo Lee
On Sun, Jun 14, 2015 at 11:34 AM, David Bremner da...@tethera.net wrote: Jinwoo Lee jinwo...@gmail.com writes: When I apply those 2 patches from you, things seem to work. I still get warnings like below, but they don't seem severe. I'm not sure if the ruby binding actually works though

[patch v3 1/4] build: integrate building ruby bindings into notmuch build process

2015-06-13 Thread Jinwoo Lee
On Sat, Jun 13, 2015 at 02:24 PM, Jinwoo Lee wrote: > That error is from 'cd bindings/ruby && ruby extconf.rb --vendor' BTW. The contents of mkmf.log are below. It shows a warning about -L/usr/local/lib and I don't have the directory /usr/local/lib on my machine. I'm not sur

[patch v3 1/4] build: integrate building ruby bindings into notmuch build process

2015-06-13 Thread Jinwoo Lee
That error is from 'cd bindings/ruby && ruby extconf.rb --vendor' BTW.

[patch v3 1/4] build: integrate building ruby bindings into notmuch build process

2015-06-13 Thread Jinwoo Lee
On Sat, Jun 13, 2015 at 02:11 PM, David Bremner wrote: > Jinwoo Lee writes: > >>> I with configure has an option to skip the ruby-bindings build. >> >> `configure' seems to try to detect whether the ruby development tools >> are installed. It thinks I have them

[patch v3 1/4] build: integrate building ruby bindings into notmuch build process

2015-06-13 Thread Jinwoo Lee
On Sat, Jun 13, 2015 at 01:46 PM, Jinwoo Lee wrote: > On Sat, Jun 13, 2015 at 12:59 PM, Jinwoo Lee wrote: >> This breaks Mac OS X. ruby-bindings depends on lib/libnotmuch.so but it >> should be lib/libnotmuch.dylib on OS X. >> >> That makes `make' and `make inst

[patch v3 1/4] build: integrate building ruby bindings into notmuch build process

2015-06-13 Thread Jinwoo Lee
On Sat, Jun 13, 2015 at 12:59 PM, Jinwoo Lee wrote: > This breaks Mac OS X. ruby-bindings depends on lib/libnotmuch.so but it > should be lib/libnotmuch.dylib on OS X. > > That makes `make' and `make install' fail. Even after I update the dependency to lib/libnotmuch.dylib, b

[patch v3 1/4] build: integrate building ruby bindings into notmuch build process

2015-06-13 Thread Jinwoo Lee
This breaks Mac OS X. ruby-bindings depends on lib/libnotmuch.so but it should be lib/libnotmuch.dylib on OS X. That makes `make' and `make install' fail. On Fri, Jun 12, 2015 at 11:37 PM, David Bremner wrote: > David Bremner writes: > >> Because ruby generates a Makefile, we have to use

Re: [patch v3 1/4] build: integrate building ruby bindings into notmuch build process

2015-06-13 Thread Jinwoo Lee
This breaks Mac OS X. ruby-bindings depends on lib/libnotmuch.so but it should be lib/libnotmuch.dylib on OS X. That makes `make' and `make install' fail. On Fri, Jun 12, 2015 at 11:37 PM, David Bremner da...@tethera.net wrote: David Bremner da...@tethera.net writes: Because ruby generates a

Re: [patch v3 1/4] build: integrate building ruby bindings into notmuch build process

2015-06-13 Thread Jinwoo Lee
On Sat, Jun 13, 2015 at 12:59 PM, Jinwoo Lee jinwo...@gmail.com wrote: This breaks Mac OS X. ruby-bindings depends on lib/libnotmuch.so but it should be lib/libnotmuch.dylib on OS X. That makes `make' and `make install' fail. Even after I update the dependency to lib/libnotmuch.dylib

Re: [patch v3 1/4] build: integrate building ruby bindings into notmuch build process

2015-06-13 Thread Jinwoo Lee
On Sat, Jun 13, 2015 at 01:46 PM, Jinwoo Lee jinwo...@gmail.com wrote: On Sat, Jun 13, 2015 at 12:59 PM, Jinwoo Lee jinwo...@gmail.com wrote: This breaks Mac OS X. ruby-bindings depends on lib/libnotmuch.so but it should be lib/libnotmuch.dylib on OS X. That makes `make' and `make install

Re: [patch v3 1/4] build: integrate building ruby bindings into notmuch build process

2015-06-13 Thread Jinwoo Lee
On Sat, Jun 13, 2015 at 02:11 PM, David Bremner da...@tethera.net wrote: Jinwoo Lee jinwo...@gmail.com writes: I with configure has an option to skip the ruby-bindings build. `configure' seems to try to detect whether the ruby development tools are installed. It thinks I have them but I

Re: [patch v3 1/4] build: integrate building ruby bindings into notmuch build process

2015-06-13 Thread Jinwoo Lee
That error is from 'cd bindings/ruby ruby extconf.rb --vendor' BTW. ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch

Re: [patch v3 1/4] build: integrate building ruby bindings into notmuch build process

2015-06-13 Thread Jinwoo Lee
On Sat, Jun 13, 2015 at 02:24 PM, Jinwoo Lee jinwo...@gmail.com wrote: That error is from 'cd bindings/ruby ruby extconf.rb --vendor' BTW. The contents of mkmf.log are below. It shows a warning about -L/usr/local/lib and I don't have the directory /usr/local/lib on my machine. I'm not sure

Is notmuchmail.org down?

2015-04-03 Thread Jinwoo Lee
I can't access the web site or the git repository since yesterday. Are we experiencing an outage? I'm not sure if this email will be delivered at all. -jinwoo ___ notmuch mailing list notmuch@notmuchmail.org

Is notmuchmail.org down?

2015-04-02 Thread Jinwoo Lee
I can't access the web site or the git repository since yesterday. Are we experiencing an outage? I'm not sure if this email will be delivered at all. -jinwoo

[PATCH v2] lib: make notmuch shared library install_name be full path on Mac OS X

2015-03-04 Thread Jinwoo Lee
On Wed, Mar 4, 2015 at 02:32 PM, "J. Lewis Muir" wrote: > The install_name of libnotmuch.dylib on Mac OS X is what is written > into a program that links against it. If it is just the name of the > shared library file, as opposed to the full path, the program won't be > able to find it when it

Re: [PATCH v2] lib: make notmuch shared library install_name be full path on Mac OS X

2015-03-04 Thread Jinwoo Lee
On Wed, Mar 4, 2015 at 02:32 PM, J. Lewis Muir jlm...@imca-cat.org wrote: The install_name of libnotmuch.dylib on Mac OS X is what is written into a program that links against it. If it is just the name of the shared library file, as opposed to the full path, the program won't be able to

No love for MacPorts?

2015-02-04 Thread Jinwoo Lee
I'm using the MacPorts version of notmuch. And I notice that it's stuck with 0.18.1. It looks like 0.18.2 was released in October and 0.19 in November. Are we not maintaining MacPorts anymore? -jinwoo

No love for MacPorts?

2015-02-04 Thread Jinwoo Lee
I'm using the MacPorts version of notmuch. And I notice that it's stuck with 0.18.1. It looks like 0.18.2 was released in October and 0.19 in November. Are we not maintaining MacPorts anymore? -jinwoo ___ notmuch mailing list notmuch@notmuchmail.org

[PATCH] emacs: Add a defcustom that specifies regexp for blocked remote images.

2015-02-02 Thread Jinwoo Lee
On Mon, Feb 2, 2015 at 02:15 PM, David Bremner wrote: > Jinwoo Lee writes: > >> And what's the process for checking the code in? I just push to the >> repo? >> >> -jinwoo > > Hi Jinwoo; > > I pushed it. We're pretty miserly with push access, but o

[PATCH] emacs: Add a defcustom that specifies regexp for blocked remote images.

2015-02-02 Thread Jinwoo Lee
And what's the process for checking the code in? I just push to the repo? -jinwoo

[PATCH] emacs: Add a defcustom that specifies regexp for blocked remote images.

2015-02-02 Thread Jinwoo Lee
It's default value is ".", meaning all remote images will be blocked by default. --- Addressed review comments. --- emacs/notmuch-show.el | 24 +--- 1 file changed, 17 insertions(+), 7 deletions(-) diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el index

[PATCH] emacs: Add a defcustom that specifies regexp for blocked remote images.

2015-02-02 Thread Jinwoo Lee
On Mon, Feb 2, 2015 at 12:32 PM, Tomi Ollila wrote: > On Mon, Feb 02 2015, Jinwoo Lee wrote: > >> It's default value is ".", meaning all remote images will be blocked >> by default. >> >> --- >> Addressed review comments. > > Ok, looks good

[PATCH] emacs: Add a defcustom that specifies regexp for blocked remote images.

2015-02-02 Thread Jinwoo Lee
Thanks for the review, guys. Sent yet another patch. BTW I'm not sure if I should specify --in-reply-to when sending updates. -jinwoo

[PATCH] emacs: Add a defcustom that specifies regexp for blocked remote images.

2015-02-02 Thread Jinwoo Lee
It's default value is ".", meaning all remote images will be blocked by default. --- Addressed review comments. --- emacs/notmuch-show.el | 27 +++ 1 file changed, 19 insertions(+), 8 deletions(-) diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el index

Re: [PATCH] emacs: Add a defcustom that specifies regexp for blocked remote images.

2015-02-02 Thread Jinwoo Lee
And what's the process for checking the code in? I just push to the repo? -jinwoo ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch

Re: [PATCH] emacs: Add a defcustom that specifies regexp for blocked remote images.

2015-02-02 Thread Jinwoo Lee
On Mon, Feb 2, 2015 at 02:15 PM, David Bremner da...@tethera.net wrote: Jinwoo Lee jinwo...@gmail.com writes: And what's the process for checking the code in? I just push to the repo? -jinwoo Hi Jinwoo; I pushed it. We're pretty miserly with push access, but once you get through

[PATCH] emacs: Add a defcustom that specifies regexp for blocked remote images.

2015-02-02 Thread Jinwoo Lee
It's default value is ., meaning all remote images will be blocked by default. --- Addressed review comments. --- emacs/notmuch-show.el | 24 +--- 1 file changed, 17 insertions(+), 7 deletions(-) diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el index

Re: [PATCH] emacs: Add a defcustom that specifies regexp for blocked remote images.

2015-02-02 Thread Jinwoo Lee
On Mon, Feb 2, 2015 at 12:32 PM, Tomi Ollila tomi.oll...@iki.fi wrote: On Mon, Feb 02 2015, Jinwoo Lee jinwo...@gmail.com wrote: It's default value is ., meaning all remote images will be blocked by default. --- Addressed review comments. Ok, looks good to me. David can perhaps amend

[PATCH] emacs: Add a defcustom that specifies regexp for blocked remote images.

2015-02-02 Thread Jinwoo Lee
It's default value is ., meaning all remote images will be blocked by default. --- Addressed review comments. --- emacs/notmuch-show.el | 27 +++ 1 file changed, 19 insertions(+), 8 deletions(-) diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el index

Re: [PATCH] emacs: Add a defcustom that specifies regexp for blocked remote images.

2015-02-02 Thread Jinwoo Lee
Thanks for the review, guys. Sent yet another patch. BTW I'm not sure if I should specify --in-reply-to when sending updates. -jinwoo ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch

[PATCH] emacs: Add a defcustom that specifies regexp for blocked remote images.

2015-01-30 Thread Jinwoo Lee
Can someone take a look please? On Thu, Jan 29, 2015 at 01:35 PM, Jinwoo Lee wrote: > It's default value is ".", meaning all remote images will be blocked > by default. > > --- > This time setting gnus-blocked-images from the correct place. > --- &

Re: [PATCH] emacs: Add a defcustom that specifies regexp for blocked remote images.

2015-01-30 Thread Jinwoo Lee
Can someone take a look please? On Thu, Jan 29, 2015 at 01:35 PM, Jinwoo Lee jinwo...@gmail.com wrote: It's default value is ., meaning all remote images will be blocked by default. --- This time setting gnus-blocked-images from the correct place. --- emacs/notmuch-show.el | 23

[PATCH] Add a defcustom for whether to block remote images by default.

2015-01-29 Thread Jinwoo Lee
On Thu, Jan 29, 2015 at 01:39 PM, Jinwoo Lee wrote: > Sent another patch. I couldn't test it with gnus-w3m though because I > don't have w3m on my machine now. I installed w3m and verified that my patch works with gnus-w3m too.

[PATCH] Add a defcustom for whether to block remote images by default.

2015-01-29 Thread Jinwoo Lee
On Thu, Jan 29, 2015 at 12:57 PM, Jinwoo Lee wrote: > On Thu, Jan 29, 2015 at 12:25 PM, Tomi Ollila wrote: >> Thanks for your contribution. You seem to have taken my suggestion >> literally just that it IIRC now only sets those when using shr renderer -- >> setting of gnus-

[PATCH] emacs: Add a defcustom that specifies regexp for blocked remote images.

2015-01-29 Thread Jinwoo Lee
It's default value is ".", meaning all remote images will be blocked by default. --- This time setting gnus-blocked-images from the correct place. --- emacs/notmuch-show.el | 23 ++- 1 file changed, 18 insertions(+), 5 deletions(-) diff --git a/emacs/notmuch-show.el

[PATCH] Add a defcustom for whether to block remote images by default.

2015-01-29 Thread Jinwoo Lee
On Thu, Jan 29, 2015 at 12:25 PM, Tomi Ollila wrote: > On Thu, Jan 29 2015, Jinwoo Lee wrote: > >> On Thu, Jan 29, 2015 at 12:58 AM, Tomi Ollila wrote: >>> On Thu, Jan 29 2015, David Bremner wrote: >>> >>>> Jinwoo Lee writes: >>>> >

Emacs: Notmuch logo is not displayed when using ELPA package

2015-01-29 Thread Jinwoo Lee
On Thu, Jan 29, 2015 at 11:49 AM, David Bremner wrote: > Jinwoo Lee writes: >> >> Yes MELPA, that is. Sorry. So it's not fixable from our side? >> > > Correct, it would only be possible to fix on the MELPA side. OK. I'll have to abandon the MELPA version then :) > > d

Emacs: Notmuch logo is not displayed when using ELPA package

2015-01-29 Thread Jinwoo Lee
On Thu, Jan 29, 2015 at 10:58 AM, David Bremner wrote: > Jinwoo Lee writes: > >> TIL notmuch shows a logo at the top of the Emacs buffer! >> >> I've been using the ELPA package so far and it didn't show the logo. I >> switched to my local git repo to test my change

Emacs: Notmuch logo is not displayed when using ELPA package

2015-01-29 Thread Jinwoo Lee
I mistakenly sent from a wrong account. Sending again... -- TIL notmuch shows a logo at the top of the Emacs buffer! I've been using the ELPA package so far and it didn't show the logo. I switched to my local git repo to test my changes and realized that we have a logo! Anyone know

privacy problem: text/html parts pull in network resources

2015-01-29 Thread Jinwoo Lee
On Thu, Jan 29, 2015 at 10:03 AM, Daniel Kahn Gillmor wrote: > On Wed 2015-01-28 18:57:25 -0500, Jinwoo Lee wrote: >> Do you mind if I add a boolean defcustom, which determines whether to >> block remote images? Its default value will be T (block), but people >> who want to

[PATCH] Add a defcustom for whether to block remote images by default.

2015-01-29 Thread Jinwoo Lee
On Thu, Jan 29, 2015 at 12:58 AM, Tomi Ollila wrote: > On Thu, Jan 29 2015, David Bremner wrote: > >> Jinwoo Lee writes: >> >>> + (shr-blocked-images (if notmuch-show-block-remote-images >>> + "." >>> +

[PATCH] emacs: Add a defcustom that specifies regexp for blocked remote images.

2015-01-29 Thread Jinwoo Lee
It's default value is ".", meaning all remote images will be blocked by default. --- emacs/notmuch-show.el | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el index 66350d4..cc6aca9 100644 --- a/emacs/notmuch-show.el

[PATCH] emacs: Add a defcustom that specifies regexp for blocked remote images.

2015-01-29 Thread Jinwoo Lee
It's default value is ., meaning all remote images will be blocked by default. --- emacs/notmuch-show.el | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el index 66350d4..cc6aca9 100644 --- a/emacs/notmuch-show.el +++

Re: [PATCH] Add a defcustom for whether to block remote images by default.

2015-01-29 Thread Jinwoo Lee
On Thu, Jan 29, 2015 at 12:58 AM, Tomi Ollila tomi.oll...@iki.fi wrote: On Thu, Jan 29 2015, David Bremner da...@tethera.net wrote: Jinwoo Lee jinwo...@gmail.com writes: + (shr-blocked-images (if notmuch-show-block-remote-images + . + shr

Emacs: Notmuch logo is not displayed when using ELPA package

2015-01-29 Thread Jinwoo Lee
I mistakenly sent from a wrong account. Sending again... -- TIL notmuch shows a logo at the top of the Emacs buffer! I've been using the ELPA package so far and it didn't show the logo. I switched to my local git repo to test my changes and realized that we have a logo! Anyone know

Re: privacy problem: text/html parts pull in network resources

2015-01-29 Thread Jinwoo Lee
On Thu, Jan 29, 2015 at 10:03 AM, Daniel Kahn Gillmor d...@fifthhorseman.net wrote: On Wed 2015-01-28 18:57:25 -0500, Jinwoo Lee wrote: Do you mind if I add a boolean defcustom, which determines whether to block remote images? Its default value will be T (block), but people who want to see

Re: Emacs: Notmuch logo is not displayed when using ELPA package

2015-01-29 Thread Jinwoo Lee
On Thu, Jan 29, 2015 at 10:58 AM, David Bremner da...@tethera.net wrote: Jinwoo Lee jinwo...@gmail.com writes: TIL notmuch shows a logo at the top of the Emacs buffer! I've been using the ELPA package so far and it didn't show the logo. I switched to my local git repo to test my changes

Re: Emacs: Notmuch logo is not displayed when using ELPA package

2015-01-29 Thread Jinwoo Lee
On Thu, Jan 29, 2015 at 11:49 AM, David Bremner da...@tethera.net wrote: Jinwoo Lee jinwo...@gmail.com writes: Yes MELPA, that is. Sorry. So it's not fixable from our side? Correct, it would only be possible to fix on the MELPA side. OK. I'll have to abandon the MELPA version

Re: [PATCH] Add a defcustom for whether to block remote images by default.

2015-01-29 Thread Jinwoo Lee
On Thu, Jan 29, 2015 at 12:25 PM, Tomi Ollila tomi.oll...@iki.fi wrote: On Thu, Jan 29 2015, Jinwoo Lee jinwo...@gmail.com wrote: On Thu, Jan 29, 2015 at 12:58 AM, Tomi Ollila tomi.oll...@iki.fi wrote: On Thu, Jan 29 2015, David Bremner da...@tethera.net wrote: Jinwoo Lee jinwo...@gmail.com

Re: [PATCH] Add a defcustom for whether to block remote images by default.

2015-01-29 Thread Jinwoo Lee
On Thu, Jan 29, 2015 at 12:57 PM, Jinwoo Lee jinwo...@gmail.com wrote: On Thu, Jan 29, 2015 at 12:25 PM, Tomi Ollila tomi.oll...@iki.fi wrote: Thanks for your contribution. You seem to have taken my suggestion literally just that it IIRC now only sets those when using shr renderer -- setting

[PATCH] emacs: Add a defcustom that specifies regexp for blocked remote images.

2015-01-29 Thread Jinwoo Lee
It's default value is ., meaning all remote images will be blocked by default. --- This time setting gnus-blocked-images from the correct place. --- emacs/notmuch-show.el | 23 ++- 1 file changed, 18 insertions(+), 5 deletions(-) diff --git a/emacs/notmuch-show.el

Re: [PATCH] Add a defcustom for whether to block remote images by default.

2015-01-29 Thread Jinwoo Lee
On Thu, Jan 29, 2015 at 01:39 PM, Jinwoo Lee jinwo...@gmail.com wrote: Sent another patch. I couldn't test it with gnus-w3m though because I don't have w3m on my machine now. I installed w3m and verified that my patch works with gnus-w3m too

[PATCH] Add a defcustom for whether to block remote images by default.

2015-01-28 Thread Jinwoo Lee
--- emacs/notmuch-show.el | 15 +++ 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el index 66350d4..bc48922 100644 --- a/emacs/notmuch-show.el +++ b/emacs/notmuch-show.el @@ -136,6 +136,11 @@ indentation." :type 'boolean

privacy problem: text/html parts pull in network resources

2015-01-28 Thread Jinwoo Lee
On Tue, Jan 27, 2015 at 08:44 PM, Jinwoo Lee wrote: > On Tue, Jan 27, 2015 at 07:47 PM, Daniel Kahn Gillmor fifthhorseman.net> wrote: >> On Sun 2015-01-25 12:51:43 -0500, David Bremner wrote: >>> Daniel Kahn Gillmor writes: >>> >>>> If i send a messa

Re: privacy problem: text/html parts pull in network resources

2015-01-28 Thread Jinwoo Lee
On Tue, Jan 27, 2015 at 08:44 PM, Jinwoo Lee jinwo...@gmail.com wrote: On Tue, Jan 27, 2015 at 07:47 PM, Daniel Kahn Gillmor d...@fifthhorseman.net wrote: On Sun 2015-01-25 12:51:43 -0500, David Bremner wrote: Daniel Kahn Gillmor d...@fifthhorseman.net writes: If i send a message

[PATCH] Add a defcustom for whether to block remote images by default.

2015-01-28 Thread Jinwoo Lee
--- emacs/notmuch-show.el | 15 +++ 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el index 66350d4..bc48922 100644 --- a/emacs/notmuch-show.el +++ b/emacs/notmuch-show.el @@ -136,6 +136,11 @@ indentation. :type 'boolean

privacy problem: text/html parts pull in network resources

2015-01-27 Thread Jinwoo Lee
On Tue, Jan 27, 2015 at 07:47 PM, Daniel Kahn Gillmor wrote: > On Sun 2015-01-25 12:51:43 -0500, David Bremner wrote: >> Daniel Kahn Gillmor writes: >> >>> If i send a message with a text/html part (either it's only text/html, >>> or all parts are rendered, or it's multipart/alternative with

Re: privacy problem: text/html parts pull in network resources

2015-01-27 Thread Jinwoo Lee
On Tue, Jan 27, 2015 at 07:47 PM, Daniel Kahn Gillmor d...@fifthhorseman.net wrote: On Sun 2015-01-25 12:51:43 -0500, David Bremner wrote: Daniel Kahn Gillmor d...@fifthhorseman.net writes: If i send a message with a text/html part (either it's only text/html, or all parts are rendered, or

[PATCH] emacs: escape % in header line format

2015-01-22 Thread Jinwoo Lee
Yup. It works! Thanks for the quick fix. Is this going to be merged to HEAD soon? -jinwoo On Thu, Jan 22, 2015 at 12:37 AM, David Bremner wrote: > We set header-line-format to the message subject, but if the subject > contains percents, the next character is interpreted as a formatting >

[PATCH] emacs: escape % in header line format

2015-01-22 Thread Jinwoo Lee
Thanks, David. But I don't think it's the correct fix. REPLACE-STRING seems to replace a string in a buffer, not a string given as a param. And it's for interactive use only. -jinwoo On Wed, Jan 21, 2015 at 11:18 PM, David Bremner wrote: > We set header-line-format to the message subject, but

Re: [PATCH] emacs: escape % in header line format

2015-01-22 Thread Jinwoo Lee
Thanks, David. But I don't think it's the correct fix. REPLACE-STRING seems to replace a string in a buffer, not a string given as a param. And it's for interactive use only. -jinwoo On Wed, Jan 21, 2015 at 11:18 PM, David Bremner da...@tethera.net wrote: We set header-line-format to the

Re: [PATCH] emacs: escape % in header line format

2015-01-22 Thread Jinwoo Lee
Yup. It works! Thanks for the quick fix. Is this going to be merged to HEAD soon? -jinwoo On Thu, Jan 22, 2015 at 12:37 AM, David Bremner da...@tethera.net wrote: We set header-line-format to the message subject, but if the subject contains percents, the next character is interpreted as a

Percent symbol is not shown in the global subject line in the Emacs notmuch-show buffer

2015-01-21 Thread Jinwoo Lee
When reading an email thread with Emacs client of notmuch, there are two places where the email subject is displayed -- one at the top of the buffer, and per each email in the thread. When the subject contains '%' in it, the global subject line at the top of the buffer doesn't show it and the

Percent symbol is not shown in the global subject line in the Emacs notmuch-show buffer

2015-01-21 Thread Jinwoo Lee
When reading an email thread with Emacs client of notmuch, there are two places where the email subject is displayed -- one at the top of the buffer, and per each email in the thread. When the subject contains '%' in it, the global subject line at the top of the buffer doesn't show it and the