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

Nikita Karetnikov nikita at karetnikov.org
Thu Jul 25 23:01:54 UTC 2013


> 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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 835 bytes
Desc: not available
URL: <http://lists.torproject.org/pipermail/tor-dev/attachments/20130726/3452749f/attachment.sig>


More information about the tor-dev mailing list