summoner icon indicating copy to clipboard operation
summoner copied to clipboard

Function "readFileText" deprecated in relude 1.1

Open Vekhir opened this issue 1 year ago • 0 comments

When building summoner using relude 1.2.0.0, the following warning occurs:

[11 of 23] Compiling Summoner.Source

src/Summoner/Source.hs:92:35: warning: [-Wdeprecations]
    In the use of ‘readFileText’
    (imported from Prelude, but defined in relude-1.2.0.0:Relude.File):
    "'readFileText' depends on the system's locale settings and can throw unexpected exceptions.
     Use 'readFileBS' instead."
   |
92 |     Local path -> catch (Just <$> readFileText path) (localError path)
   |                                   ^^^^^^^^^^^^

The relude Changelog mentions this change in v1.1.0.0 and also suggests a fix:

Use readFileBS / readFileLBS instead to convert to Text / LText / String using decodeUtf8 and similar functions.

With that change, the supported version for relude can be bumped to 1.2.0.0

Vekhir avatar Sep 12 '23 14:09 Vekhir