On Mon, Nov 7, 2016 at 2:03 AM, Guillaume Lelarge <guillaume@lelarge.info> wrote: >> >> Agreed. I was afraid of that, but for some reason, didn't find that. I'll >> fix this. > > > Fixed in v4. >
This fix is broken.
70 - if (dopt.include_everything && !dopt.schemaOnly) 71 + if (dopt.include_everything && !dopt.schemaOnly && !dopt.outputBlobs) 72 dopt.outputBlobs = true;
dopt.outputBlobs set to FALSE when option -B is specified and this IF condition reverts to TRUE which force to dump blobs.
I don't see what you mean. It forces dump of Blobs if we didn't use -B and if we include everything in the dump, which seems good to me. What did you try that didn't work as expected?
>> >> >>> >>> #2 : >>> We should add note for default behaviour if --no-blobs & --blobs both >>> are specified. >>> >> >> Right. I don't know how I will handle this, but you're right that the >> behaviour should be specified. I'll also fix this. >> > > I checked other options, such as --format, and there's nothing noted as a > default behaviour. Last one wins, which is what this patch does. >
Such note exists for --table & --exclude-table option, see following lines in pg_dump.sgml
569 <para> 570 When both <option>-t</> and <option>-T</> are given, the behavior 571 is to dump just the tables that match at least one <option>-t</> 572 switch but no <option>-T</> switches. If <option>-T</> appears 573 without <option>-t</>, then tables matching <option>-T</> are 574 excluded from what is otherwise a normal dump. 575 </para> 576 </listitem>