Opened 11 years ago
Closed 11 years ago
#37923 closed enhancement (wontfix)
MacPorts: add variant to use MacPorts ports as dependencies
| Reported by: | cooljeanius (Eric Gallager) | Owned by: | macports-tickets@… |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | 2.1.2 |
| Keywords: | haspatch | Cc: | raimue (Rainer Müller), ryandesign (Ryan Carsten Schmidt) |
| Port: | MacPorts |
Description
Attached is a diff to apply to the Portfile for the MacPorts port. I've linted and tested it and it works.
Attachments (2)
Change History (7)
Changed 11 years ago by cooljeanius (Eric Gallager)
| Attachment: | Portfile.diff added |
|---|
comment:1 follow-up: 2 Changed 11 years ago by ryandesign (Ryan Carsten Schmidt)
Why would we want this?
comment:2 Changed 11 years ago by cooljeanius (Eric Gallager)
Replying to ryandesign@…:
Why would we want this?
Well, personally, I want it so that I can be sure that my MacPorts installation is taking advantage of all the latest and greatest things available to it. I bet some other users might want this option, too.
comment:3 follow-up: 4 Changed 11 years ago by raimue (Rainer Müller)
| Cc: | raimue@… added |
|---|
The MacPorts port is currently only meant to be used for building the MacPorts .dmg and .pkg installers. Maybe it shouldn't even be in the tree, but rather live in base/portmgr.
While this proposed patch is a nice experiment on how MacPorts might become self-contained, I don't think this is something that should be done right now and offered to end-users. At the moment, we do not have any fallbacks in case one of the dependencies breaks due to bad linking.
What exactly is the intent of this? It only seems to replace the binaries called by MacPorts and does not cover libraries linked to Pextlib or the registry (curl and sqlite3).
Changed 11 years ago by cooljeanius (Eric Gallager)
| Attachment: | Portfile.2.diff added |
|---|
new diff to apply
comment:4 Changed 11 years ago by cooljeanius (Eric Gallager)
Replying to raimue@…:
The MacPorts port is currently only meant to be used for building the MacPorts .dmg and .pkg installers. Maybe it shouldn't even be in the tree, but rather live in base/portmgr.
Fair enough. I was actually thinking the contrib folder, but that works too.
It only seems to replace the binaries called by MacPorts and does not cover libraries linked to Pextlib or the registry (curl and sqlite3).
curl and sqlite3 added in the latest version. I did just plain sqlite3 instead of tcl-sqlite3 because the latter is broken by #37376. I also left out the --with-tcl-sqlite3=/opt/local/lib/sqlite3.7.15.1/ flag though because that version number at the end of the directory there looks like it would change often, and I'm not sure how I'd check for that...
comment:5 Changed 11 years ago by ryandesign (Ryan Carsten Schmidt)
| Cc: | ryandesign@… added |
|---|---|
| Resolution: | → wontfix |
| Status: | new → closed |
| Summary: | Submission: New variants for the MacPorts port: Allow ports provided by MacPorts to fulfill missing dependencies for it → MacPorts: add variant to use MacPorts ports as dependencies |
| Type: | submission → enhancement |
I really don't think we want any such patch, since the only reason this port exists is so that we can create installer packages of new versions of MacPorts (no user should ever install the MacPorts port directly), and in the installer packages, the MacPorts installation must be self-contained; it cannot be linked with MacPorts-provided libraries that most users who install the disk image won't already have installed.
diff to apply to the Portfile for the MacPorts port