Skip to content

duck: update to 9.2.4.43667 #29985

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

duck: update to 9.2.4.43667 #29985

wants to merge 2 commits into from

Conversation

@trustbyte
Copy link
Contributor

@trustbyte trustbyte commented Nov 8, 2025 •
edited
Loading

Description

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

macOS 26.1 25B78 arm64
Command Line Tools 26.1.0.0.1.1761104275

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?
  • 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?
@macportsbot
Copy link

macportsbot commented Nov 8, 2025

Notifying maintainers:
@JacksonIsaac for port duck.
@trustbyte
Copy link
Contributor Author

trustbyte commented Nov 8, 2025

How to check the rev-upgrade part on this new release?
@reneeotten
Copy link
Contributor

reneeotten commented Nov 9, 2025

How to check the rev-upgrade part on this new release?

what do you mean by "check the rev-upgrade part". If there is a rev-upgrade after you install something it likely means that there was a change in library name somewhere and a dependent is now broken. That's typically just solved by increasing the revision of the dependent. You shouldn't see this when just updating a port...
@trustbyte
Copy link
Contributor Author

trustbyte commented Nov 9, 2025

Sorry, I should have mentioned that there is a comment inside the file that has something added that should be checked at upgrade..
rev-upgrade is mentioned.
@barracuda156
Copy link
Contributor

barracuda156 commented Nov 15, 2025

This port should be built from source, but install an opaque pre-built stuff, there is no justification for that.
@trustbyte
Copy link
Contributor Author

trustbyte commented Nov 15, 2025

Okey, is it this, right? https://github.com/iterate-ch/cyberduck
Building with mvn/ant? Oh boy, oh boy. Let's see how it goes.

Draft.
@trustbyte trustbyte marked this pull request as draft November 15, 2025 20:36
@trustbyte
Copy link
Contributor Author

trustbyte commented Nov 17, 2025

Any ideas how to fix DerivedData?
I tried quite a few, none worked on my machine..
@barracuda156
Copy link
Contributor

barracuda156 commented Nov 18, 2025

@trustbyte If I had a working portfile, I would have just referred to it. I didn’t try building this yet, since it uses Java, and on powerpc that would be tricky, if possible at all (we only got headless JDK atm). I also got no experience with Java builds on x86.
barracuda156
barracuda156 reviewed Nov 22, 2025
View reviewed changes
net/duck/Portfile
build.cmd xcodebuild
build {
set env(HOME) ${workpath}
set env(DEVELOPER_DIR) /Applications/Xcode.app/Contents/Developer
Copy link
Contributor

@barracuda156 barracuda156 Nov 22, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This holds only for some macOS versions. I don’t think it should be hardcoded in the portfile, you can pick the value from macports.conf.
barracuda156
barracuda156 reviewed Nov 22, 2025
View reviewed changes
net/duck/Portfile
set env(DEVELOPER_DIR) /Applications/Xcode.app/Contents/Developer

system -W ${worksrcpath} \
"/Applications/Xcode.app/Contents/Developer/usr/bin/xcodebuild \
Copy link
Contributor

@barracuda156 barracuda156 Nov 22, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This path also gonna be wrong, when developer_dir is elsewhere.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainer: open Affects an openmaintainer port type: enhancement type: update

5 participants

@trustbyte @macportsbot @reneeotten @barracuda156 @JacksonIsaac