Skip to content

TPF: drop support#8378

Closed
bagder wants to merge 2 commits into
masterfrom
bagder/rm-tpf
Closed

TPF: drop support#8378
bagder wants to merge 2 commits into
masterfrom
bagder/rm-tpf

Conversation

@bagder

@bagder bagder commented Feb 3, 2022

Copy link
Copy Markdown
Member

There has been no TPF related changes done since September 2010 (commit
7e1a45e) and since this is a platform that is relatively different
than many others (== needs attention), I draw the conclusion that this
build is broken since a long time.

@bagder bagder added the tidy-up label Feb 3, 2022
@monnerat

monnerat commented Feb 3, 2022

Copy link
Copy Markdown
Contributor

If you do that, this is also an open door for all the CURL_DOES_CONVERSIONS stuff removal, as it seems this is the only platform using it.
Just my 2 cents!

@bagder

bagder commented Feb 3, 2022

Copy link
Copy Markdown
Member Author

Oh wow, thanks for highlighting this fact. I think that's actually also another strong reason to suspect that the TPF build doesn't work anymore as the conversion stuff is fairly complicated and easy to mess up.

Also, a fairly big gain to be able to rip that out as this define complicates the logic in many places.

@monnerat

monnerat commented Feb 3, 2022

Copy link
Copy Markdown
Contributor

Big slashing, isn't it ! :-)

@bagder

bagder commented Feb 3, 2022

Copy link
Copy Markdown
Member Author

Yes, that is a significant chunk. There might even be some more to fix after this when some flows have been simplified.

Comment thread lib/setopt.c Outdated
There has been no TPF related changes done since September 2010 (commit
7e1a45e) and since this is a platform that is relatively different
than many others (== needs attention), I draw the conclusion that this
build is broken since a long time.

Closes #8378
TPF was the only user and support for that was dropped.

Closes #8378
@bagder bagder closed this in da15443 Feb 4, 2022
bagder added a commit that referenced this pull request Feb 4, 2022
TPF was the only user and support for that was dropped.

Closes #8378
@bagder bagder deleted the bagder/rm-tpf branch February 4, 2022 07:06
vszakats added a commit to vszakats/curl that referenced this pull request Sep 8, 2024
vszakats added a commit that referenced this pull request Sep 9, 2024
pps83 pushed a commit to pps83/curl that referenced this pull request Apr 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants