Hi,

Thanks for the clarification. I guess porting GCC to HelenOS will also
require libposix and since GNU binutils have already been ported, the
work will be somewhat easier in the sense one can have a good
headstart. Complete port of GCC may also require changes and addition
in libposix and binutils. This is all completely based on guesswork
and requires closer inspection to determine the actual implementation
details, which i intend do from now onward.

Yes, I believe that these initial observations are quite correct.

Any pointers and
suggestions regarding it will be very welcome :-)

The porting of GCC might be really quite similar to porting binutils, therefore I suggest that you read the binutils port maintenance guide [1] and perhaps also the relevant emails in the archive of this mailing list from Petr Koupy ([email protected]) from last year's GSoC. They might give you some idea what problems Petr had to solve when porting binutils.

[1] http://trac.helenos.org/wiki/BinutilsMaintenance

I have attached a patch of the touch.c file, where i have added -c option.

I'll use this opportunity to improve the touch.c based on your patch, but in a larger extent. Do you give us the permission to integrate your patch into our source tree under the BSD license?

Regarding your patch, it is mostly OK. I have only some minor comments:

* You should have altered the initial TODO comment in touch.c, too. You
  have implemented one of the TODO items.

* Line continuations (when forced to break the line when reaching 80
  characters) should be indented by four spaces. I admit that the
  original touch.c did not follow this rule perfectly.

* Identifiers should be named in a better way. Instead of "opt_ind"
  (too similar to "optind") you should rather use "longind". Instead of
  "cflag" you should rather use "no_create".

Anyway, thanks for your contribution!


M.D.

_______________________________________________
HelenOS-devel mailing list
[email protected]
http://lists.modry.cz/cgi-bin/listinfo/helenos-devel

Reply via email to