[Git][ghc/ghc][wip/T21976] 5 commits: testsuite: 21651 add test for closeFdWith + setNumCapabilities

Ben Gamari (@bgamari) gitlab at gitlab.haskell.org
Wed Aug 10 13:52:23 UTC 2022



Ben Gamari pushed to branch wip/T21976 at Glasgow Haskell Compiler / GHC


Commits:
76b52cf0 by Douglas Wilson at 2022-08-10T06:01:53-04:00
testsuite: 21651 add test for closeFdWith + setNumCapabilities

This bug does not affect windows, which does not use the
base module GHC.Event.Thread.

- - - - -
7589ee72 by Douglas Wilson at 2022-08-10T06:01:53-04:00
base: Fix races in IOManager (setNumCapabilities,closeFdWith)

Fix for #21651

Fixes three bugs:

- writes to eventManager should be atomic. It is accessed concurrently by ioManagerCapabilitiesChanged and closeFdWith.
- The race in closeFdWith described in the ticket.
- A race in getSystemEventManager where it accesses the 'IOArray' in
  'eventManager' before 'ioManagerCapabilitiesChanged' has written to
  'eventManager', causing an Array Index exception. The fix here is to
  'yield' and retry.

- - - - -
dc76439d by Trevis Elser at 2022-08-10T06:02:28-04:00
Updates language extension documentation

Adding a 'Status' field with a few values:
- Deprecated
- Experimental
- InternalUseOnly
- Noting if included in 'GHC2021', 'Haskell2010' or 'Haskell98'

Those values are pulled from the existing descriptions or elsewhere in
the documentation.

While at it, include the :implied by: where appropriate, to provide
more detail.

Fixes #21475

- - - - -
823fe5b5 by Jens Petersen at 2022-08-10T06:03:07-04:00
hadrian RunRest: add type signature for stageNumber

avoids warning seen on 9.4.1:

src/Settings/Builders/RunTest.hs:264:53: warning: [-Wtype-defaults]
    • Defaulting the following constraints to type ‘Integer’
        (Show a0)
          arising from a use of ‘show’
          at src/Settings/Builders/RunTest.hs:264:53-84
        (Num a0)
          arising from a use of ‘stageNumber’
          at src/Settings/Builders/RunTest.hs:264:59-83
    • In the second argument of ‘(++)’, namely
        ‘show (stageNumber (C.stage ctx))’
      In the second argument of ‘($)’, namely
        ‘"config.stage=" ++ show (stageNumber (C.stage ctx))’
      In the expression:
        arg $ "config.stage=" ++ show (stageNumber (C.stage ctx))
    |
264 |             , arg "-e", arg $ "config.stage="    ++ show (stageNumber (C.stage ctx))
    |                                                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

compilation tested locally

- - - - -
b2faa987 by Ben Gamari at 2022-08-10T09:07:15-04:00
hadrian: Don't attempt to install documentation if doc/ doesn't exist

Previously we would attempt to install documentation even if the `doc`
directory doesn't exist (e.g. due to `--docs=none`). This would result
in the surprising side-effect of the entire contents of the bindist
being installed in the destination documentation directory. Fix this.

Fixes #21976.

- - - - -


30 changed files:

- docs/users_guide/bugs.rst
- docs/users_guide/exts/binary_literals.rst
- docs/users_guide/exts/constrained_class_methods.rst
- docs/users_guide/exts/constraint_kind.rst
- docs/users_guide/exts/datatype_contexts.rst
- docs/users_guide/exts/deriving_extra.rst
- docs/users_guide/exts/duplicate_record_fields.rst
- docs/users_guide/exts/empty_case.rst
- docs/users_guide/exts/empty_data_deriving.rst
- docs/users_guide/exts/existential_quantification.rst
- docs/users_guide/exts/explicit_forall.rst
- docs/users_guide/exts/explicit_namespaces.rst
- docs/users_guide/exts/ffi.rst
- docs/users_guide/exts/field_selectors.rst
- docs/users_guide/exts/flexible_contexts.rst
- docs/users_guide/exts/functional_dependencies.rst
- docs/users_guide/exts/gadt_syntax.rst
- docs/users_guide/exts/generics.rst
- docs/users_guide/exts/hex_float_literals.rst
- docs/users_guide/exts/import_qualified_post.rst
- docs/users_guide/exts/instances.rst
- docs/users_guide/exts/kind_signatures.rst
- docs/users_guide/exts/let_generalisation.rst
- docs/users_guide/exts/linear_types.rst
- docs/users_guide/exts/multi_param_type_classes.rst
- docs/users_guide/exts/newtype_deriving.rst
- docs/users_guide/exts/nk_patterns.rst
- docs/users_guide/exts/nullary_type_classes.rst
- docs/users_guide/exts/nullary_types.rst
- docs/users_guide/exts/numeric_underscores.rst


The diff was not included because it is too large.


View it on GitLab: https://gitlab.haskell.org/ghc/ghc/-/compare/e754f7ce76545ef9af12e247ed4764ee29fdf787...b2faa987c39bf344f5a4caba7c146cf01c00c306

-- 
View it on GitLab: https://gitlab.haskell.org/ghc/ghc/-/compare/e754f7ce76545ef9af12e247ed4764ee29fdf787...b2faa987c39bf344f5a4caba7c146cf01c00c306
You're receiving this email because of your account on gitlab.haskell.org.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.haskell.org/pipermail/ghc-commits/attachments/20220810/6c18e2e0/attachment.html>


More information about the ghc-commits mailing list