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

Re: [tor-dev] PRELIMINARY: [PATCH 3/3] Replace 'TorDNSEL.System.Timeout' with 'System.Timeout'.



> Using `git blame` or `git log -S` or `git log -G` will actually work
> better than a file by file summary. If you throw in the `-M` option,
> it'll work accross renames, for examples.

OK.  But that information will be lost if we decide to change a VCS for
some reason.

> Why not put what you wrote in your description email in there? This was
> a good explaination of why the change was actually needed! :)

>> "GHC was recently changed to not allow you to use newtypes in FFI
>> imports unless the constructor of the newtype is in scope." [1]
>> 
>> [1] http://ghc.haskell.org/trac/ghc/ticket/5610

OK.  So let me summarize:

  1. I shouldn't explicitly name changed functions or modules.

  2. I can add a small description (like the above) if it's appropriate.

Did I forget anything?

> I am looking forward to the moment you'll have a patch set that
> builds.

It won't happen overnight.  Right now I can only build eight modules.
At least sixteen modules need adjustments.  Not very impressive, yep.

Attachment: pgp9fo2kyxB9t.pgp
Description: PGP signature

_______________________________________________
tor-dev mailing list
tor-dev@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-dev