Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

misc: remove support for curl_off_t < 8 bytes #10597

Closed
wants to merge 2 commits into from

Conversation

bagder
Copy link
Member

@bagder bagder commented Feb 23, 2023

No description provided.

@bagder bagder closed this in 8356826 Feb 24, 2023
@bagder bagder deleted the bagder/require-64bit-datatype branch February 24, 2023 16:07
bch pushed a commit to bch/curl that referenced this pull request Jul 19, 2023
vszakats added a commit that referenced this pull request Jan 10, 2025
curl requires C compilers supporting 64-bit `curl_off_t` type since
8356826 #10597 (v8.0.0).

Assume the MSVC compiler offers the necessary support.

It makes curl require Visual Studio .NET 2003, v7.1 (`_MSC_VER = 1310`).
With the possibility that 1300 (Visual Studio .NET, v7.0, 2002), or 1200
(Visual C++, 32-bit, v6.0, 1998) may also work.

Follow-up to ca18198 #15952
Closes #15955
vszakats added a commit that referenced this pull request Jan 10, 2025
- DJGPP 1.x (including `__GO32__`) (MS-DOS)
  DJGPP 2.x support remains unchanged.
- Salford C (Windows)
- Turbo C (Windows 16-bit)
- Borland C++ < 5.2 (Windows 16-bit?)
- Pelles C < 2.8 (Windows)

These targets mapped `curl_off_t` to `long`. On Windows and MS-DOS
`long` is always 32-bit.

curl requires C compilers supporting 64-bit `curl_off_t` type since
8356826 #10597 (v8.0.0).

Also: drop remaining `__GO32__` and Salford C guards.

Closes #15957
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants