<div dir="ltr">I didn't read closely enough: I see now that it's a warning, not an error. I initially didn't export that constructor since it's only present on Windows for API compatibility, but will never be used. Since this is just the internals module, I can export it, but my preference would in fact be to leave it as-is with the warning. Two alternatives:<div><br></div><div>1. Create a new hidden module that creates and exports the type constructor, just to hide the warning. I'm -1 on that, since that's extra compile time everyone has to endure just for warning avoidance.</div><div>2. base could export CGid for Windows (currently, it does not).</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 2, 2015 at 5:17 AM, Michael Snoyman <span dir="ltr"><<a href="mailto:michael@snoyman.com" target="_blank">michael@snoyman.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I'll look into this. I just made a new release of process, and was certain I tested on Windows, but perhaps something changed between that commit and release.</div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Mon, Nov 2, 2015 at 5:15 AM, Simon Peyton Jones <span dir="ltr"><<a href="mailto:simonpj@microsoft.com" target="_blank">simonpj@microsoft.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">
<div lang="EN-GB" link="#0563C1" vlink="#954F72">
<div>
<p class="MsoNormal">I’m getting this on HEAD in te ‘<u></u><u></u></p>
<p>libraries\process\System\Process\Internals.hs:106:16: warning:<u></u><u></u></p>
<p> Defined but not used: data constructor ‘CGid’<u></u><u></u></p>
<p class="MsoNormal">Indeed it looks as if CGId(..) should be exported, else createProcess is unusuable. This looks like the right change. Would someone like to check and make the change<u></u><u></u></p>
<p class="MsoNormal">Simon<u></u><u></u></p>
<p>diff --git a/System/Process/Internals.hs b/System/Process/Internals.hs<u></u><u></u></p>
<p>index 5575ac4..3e23ad5 100644<u></u><u></u></p>
<p>--- a/System/Process/Internals.hs<u></u><u></u></p>
<p>+++ b/System/Process/Internals.hs<u></u><u></u></p>
<p>@@ -37,6 +37,8 @@ module System.Process.Internals (<u></u><u></u></p>
<p>#if !defined(mingw32_HOST_OS) && !defined(__MINGW32__)<u></u><u></u></p>
<p> pPrPr_disableITimers, c_execvpe,<u></u><u></u></p>
<p> ignoreSignal, defaultSignal,<u></u><u></u></p>
<p>+#else<u></u><u></u></p>
<p>+ CGid(..), GroupID, UserID,<u></u><u></u></p>
<p>#endif<u></u><u></u></p>
<p> withFilePathException, withCEnvironment,<u></u><u></u></p>
<p> translate,<span style="color:windowtext"><u></u><u></u></span></p>
</div>
</div>
<br></div></div>_______________________________________________<br>
ghc-devs mailing list<br>
<a href="mailto:ghc-devs@haskell.org" target="_blank">ghc-devs@haskell.org</a><br>
<a href="http://mail.haskell.org/cgi-bin/mailman/listinfo/ghc-devs" rel="noreferrer" target="_blank">http://mail.haskell.org/cgi-bin/mailman/listinfo/ghc-devs</a><br>
<br></blockquote></div><br></div>
</blockquote></div><br></div>