Opened 12 years ago

Closed 12 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)

Portfile.diff (3.8 KB) - added by cooljeanius (Eric Gallager) 12 years ago.
diff to apply to the Portfile for the MacPorts port
Portfile.2.diff (4.6 KB) - added by cooljeanius (Eric Gallager) 12 years ago.
new diff to apply

Download all attachments as: .zip

Change History (7)

Changed 12 years ago by cooljeanius (Eric Gallager)

Attachment: Portfile.diff added

diff to apply to the Portfile for the MacPorts port

comment:1 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Why would we want this?

comment:2 in reply to:  1 Changed 12 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 Changed 12 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 12 years ago by cooljeanius (Eric Gallager)

Attachment: Portfile.2.diff added

new diff to apply

comment:4 in reply to:  3 Changed 12 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 12 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: ryandesign@… added
Resolution: wontfix
Status: newclosed
Summary: Submission: New variants for the MacPorts port: Allow ports provided by MacPorts to fulfill missing dependencies for itMacPorts: add variant to use MacPorts ports as dependencies
Type: submissionenhancement

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.

Note: See TracTickets for help on using tickets.