Thanks for your your patch! Some comments: Darshit Shah <dar...@gmail.com> writes:
> Attaching a patch to make wget abort instead of sending an empty request if > the file pointed to in --post-file does not exist. + return FILEBADFILE; what exit status will be returned by the wget process? I am not sure FILEBADFILE is handled by get_exit_status (in exits.c). I think we should ensure the exit status is 3 in this case (File I/O error). As additional bonus, could you write a simple test case to exercise your code? You can copy the Test-noop.px file as a starting point then change the cmdline to something like (add the new file to the Makefile.am file too): my $cmdline = $WgetTest::WGETPATH . " --post-file=nothere http://localhost:{{port}}/"; Then you can check that the expected_error_code is 3. Cheers, Giuseppe