Re: [HACKERS] [trivial patch] typo in doc/src/sgml/sepgsql.sgml

2012-02-16 Thread Dimitri Fontaine
Alvaro Herrera alvhe...@commandprompt.com writes:
 If you copied from a pager, those tend to expand tabs to spaces, so the
 patch gets mangled at that point.  At least less does that.  OTOH if
 you :r the patch file, it works fine.

You should be able to add an “inline” attachment too, and the receiver
should then be able to both see it inline in the email and store it as a
file.  Tom uses inline attachments a lot, and apparently very few MUA
are allowing both features on that (gnus is perfectly fine here).

Regards,
-- 
Dimitri Fontaine
http://2ndQuadrant.fr PostgreSQL : Expertise, Formation et Support

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] [trivial patch] typo in doc/src/sgml/sepgsql.sgml

2012-02-15 Thread Robert Haas
On Wed, Feb 15, 2012 at 5:38 AM, Christoph Berg c...@df7cb.de wrote:
 diff --git a/doc/src/sgml/sepgsql.sgml b/doc/src/sgml/sepgsql.sgml
 index e45c258..ee0a255 100644
 *** a/doc/src/sgml/sepgsql.sgml
 --- b/doc/src/sgml/sepgsql.sgml
 *** UPDATE t1 SET x = 2, y = md5sum(y) WHERE
 *** 358,364 
  /synopsis

      In this case we must have literaldb_table:select/ in addition to
 !     literaldb_table:update/, because literalt1.a/ is referenced
      within the literalWHERE/ clause.  Column-level permissions will also 
 be
      checked for each referenced column.
     /para
 --- 358,364 
  /synopsis

      In this case we must have literaldb_table:select/ in addition to
 !     literaldb_table:update/, because literalt1.z/ is referenced
      within the literalWHERE/ clause.  Column-level permissions will also 
 be
      checked for each referenced column.
     /para

Fixed, but note that I had to recreate the patch by manual
examination. Including it inline tends to garble things.

 (It is unclear to me why the same example is cited twice here, but the
 text around them is consistent with that.)

Fixed this too, and did some related rewording and proofreading.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] [trivial patch] typo in doc/src/sgml/sepgsql.sgml

2012-02-15 Thread Christoph Berg
Re: Robert Haas 2012-02-15 
ca+tgmozjutszhpxwwzcm4kjjwh__1admo3kosbbq+fhscqp...@mail.gmail.com
 Fixed, but note that I had to recreate the patch by manual
 examination. Including it inline tends to garble things.

Hmm, I thought I had :set paste and everything...

  (It is unclear to me why the same example is cited twice here, but the
  text around them is consistent with that.)
 
 Fixed this too, and did some related rewording and proofreading.

Makes much more sense now. Thanks!

Christoph
-- 
c...@df7cb.de | http://www.df7cb.de/


signature.asc
Description: Digital signature


Re: [HACKERS] [trivial patch] typo in doc/src/sgml/sepgsql.sgml

2012-02-15 Thread Alvaro Herrera

Excerpts from Christoph Berg's message of mié feb 15 12:16:52 -0300 2012:
 Re: Robert Haas 2012-02-15 
 ca+tgmozjutszhpxwwzcm4kjjwh__1admo3kosbbq+fhscqp...@mail.gmail.com
  Fixed, but note that I had to recreate the patch by manual
  examination. Including it inline tends to garble things.
 
 Hmm, I thought I had :set paste and everything...

If you copied from a pager, those tend to expand tabs to spaces, so the
patch gets mangled at that point.  At least less does that.  OTOH if
you :r the patch file, it works fine.

-- 
Álvaro Herrera alvhe...@commandprompt.com
The PostgreSQL Company - Command Prompt, Inc.
PostgreSQL Replication, Consulting, Custom Development, 24x7 support

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] [trivial patch] typo in doc/src/sgml/sepgsql.sgml

2012-02-15 Thread Robert Haas
On Wed, Feb 15, 2012 at 1:45 PM, Alvaro Herrera
alvhe...@commandprompt.com wrote:
 Excerpts from Christoph Berg's message of mié feb 15 12:16:52 -0300 2012:
 Re: Robert Haas 2012-02-15 
 ca+tgmozjutszhpxwwzcm4kjjwh__1admo3kosbbq+fhscqp...@mail.gmail.com
  Fixed, but note that I had to recreate the patch by manual
  examination. Including it inline tends to garble things.

 Hmm, I thought I had :set paste and everything...

 If you copied from a pager, those tend to expand tabs to spaces, so the
 patch gets mangled at that point.  At least less does that.  OTOH if
 you :r the patch file, it works fine.

It is also possible that my email client is to blame.  All I know for
sure is I had to regenerate the patch.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers