Hi Amul,
--
2016-11-04 7:52 GMT+01:00 amul sul <sulamul@xxxxxxxxx>:
Hi Guillaume,
I found following issues with this patch, sorry missed in previous post:
You don't have to be sorry for me doing shitty things :)
#1 :
43 @@ -392,6 +393,10 @@ main(int argc, char **argv)
44 dopt.outputBlobs = true;
45 break;
46
47 + case 'B': /* Don't dump blobs */
48 + dopt.include_everything = false;
49 + break;
50 +
Touching dopt.include_everything flag does not seems to be a good idea
for '--no-blobs' option, our intension is to exclude blob only, but
this excluds other dump too (e.g COMMENT ON DATABASE, CREATE
EXTENSION, COMMENT ON EXTENSION, .., etc) that what we don't want,
right?
Agreed. I was afraid of that, but for some reason, didn't find that. I'll fix this.
#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'll try to work on this today but as I'm in pgconf.eu 2016, it may be only for tomorrow.
Thank you.
--
