orbit-idl-c-backend.c:out_for_pass() really that bad?



Hi,

At line 130 a directory ".deps" is created, presumably because it is
expected that rinfo->deps_file is an absolute path prefixed with
"./.deps/".

This is, IMHO, broken. rinfo->deps_file should be regarded as the
absolute path to the deps file without any assumption on the directory
part.

I have a trivial patch that fixes this by simply removing the mkdir
logic. It naturally assumes that any directory part of rinfo->deps_file
already exists. Will this patch be considered ?


-- 
  jules



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]