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

bctoolbox, ortp: fix build for < 10.7 #25341

Merged
merged 2 commits into from
Sep 21, 2024
Merged

Conversation

barracuda156
Copy link
Contributor

Description

Unbreak both ports for old systems.

Type(s)
  • bugfix
  • enhancement
  • security fix
Tested on

macOS 10.6
Xcode 3.2

Verification

Have you

  • followed our Commit Message Guidelines?
  • squashed and minimized your commits?
  • checked that there aren't other open pull requests for the same change?
  • referenced existing tickets on Trac with full URL in commit message?
  • checked your Portfile with port lint --nitpick?
  • tried existing tests with sudo port test?
  • tried a full install with sudo port -vst install?
  • tested basic functionality of all binary files?
  • checked that the Portfile's most important variants haven't been broken?

@barracuda156
Copy link
Contributor Author

What is this??

Installing MacPorts
  curl: (28) Failed to connect to github.com port 443 after 75006 ms: Couldn't connect to server
  Fetching base failed: 0
  Extracting...
  tar: Error opening archive: Failed to open 'MacPorts-2.9.3-21.tar.bz2'

@markemer
Copy link
Member

I mean, it doesn't break the 12, 13, and 14 builds. I've got no way to test if this works. If no objections come in soon, I'll merge it as it's no maintainer. Also, @barracuda156 if you want to take the maintainership over, feel free. (hint, hint 😄)

@reneeotten
Copy link
Contributor

@barracuda156 did upstream agree and/or is willing to accept these patches?

@barracuda156
Copy link
Contributor Author

barracuda156 commented Aug 15, 2024

@barracuda156 did upstream agree and/or is willing to accept these patches?

@reneeotten The only experimental patch here is a conditional dropping of redundant framework. How do you suggest it to be submitted to upstream?

P. S. There is no need to get a permission of upstream to fix their bug, like a missing header.

@reneeotten
Copy link
Contributor

@barracuda156 did upstream agree and/or is willing to accept these patches?

@reneeotten The only experimental patch here is a conditional dropping of redundant framework. How do you suggest it to be submitted to upstream?

P. S. There is no need to get a permission of upstream to fix their bug, like a missing header.

Sure, you don't need permission from anyone and that is of course a correct change. But how do you expect me or anyone else to vet all patches you add here (or in other PRs for that matter). I see that you submitted them upstream to bctoolbox but they haven't responded. The patch-legacy.diff for ortp seems to have not been submitted -or at least there is no URL-. I don't know what it does and whether or not that is correct; that's the whole idea of sending it upstream and see if they agree with it.

@markemer
Copy link
Member

I mean, anything for systems like 10.6/10.7 might but could be unlikely to be upstreamed.

@barracuda156
Copy link
Contributor Author

barracuda156 commented Aug 23, 2024

The patch-legacy.diff for ortp seems to have not been submitted -or at least there is no URL-.

That one I can submit.

UPD. Done: BelledonneCommunications/ortp#22
Let’s see, maybe upstream suggest something better instead.

@barracuda156
Copy link
Contributor Author

@reneeotten Bump

@reneeotten reneeotten merged commit f01d023 into macports:master Sep 21, 2024
3 checks passed
@barracuda156 barracuda156 deleted the ortp branch September 21, 2024 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants