Skip to content

test: add repro for #9538 (subst on 32-bit)#9539

Merged
emillon merged 1 commit intomainfrom
repro-9538
Jan 23, 2024
Merged

test: add repro for #9538 (subst on 32-bit)#9539
emillon merged 1 commit intomainfrom
repro-9538

Conversation

@emillon
Copy link
Copy Markdown
Collaborator

@emillon emillon commented Dec 19, 2023

No description provided.

Signed-off-by: Etienne Millon <me@emillon.org>
@emillon emillon merged commit 7f770e0 into main Jan 23, 2024
@emillon emillon deleted the repro-9538 branch January 23, 2024 11:58
emillon added a commit to emillon/dune that referenced this pull request Feb 5, 2024
Signed-off-by: Etienne Millon <me@emillon.org>
@emillon emillon mentioned this pull request Feb 5, 2024
emillon added a commit to emillon/dune that referenced this pull request Feb 5, 2024
Signed-off-by: Etienne Millon <me@emillon.org>
emillon added a commit that referenced this pull request Feb 5, 2024
* test: add repro for #9538 (subst on 32-bit) (#9539)

Signed-off-by: Etienne Millon <me@emillon.org>

* refactor: detect large files in Io functions (#9828)

`Io.read_all` and related functions read the contents of a file in a
string, which has a size limit (`Sys.max_string_length`) and can be an
issue in 32-bit systems. This makes an explicit check and raises a
`Code_error` in these situations.

Signed-off-by: Etienne Millon <me@emillon.org>

* fix(subst): ignore large files (#9811)

Fixes #9538

This logs a warning for large files (>16MB on 32-bit systems).

Signed-off-by: Etienne Millon <me@emillon.org>

---------

Signed-off-by: Etienne Millon <me@emillon.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant