Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#15881 closed defect (fixed)

Building foo with target all

Reported by: ryandesign (Ryan Carsten Schmidt) Owned by: macports-tickets@…
Priority: Normal Milestone: MacPorts 1.7.0
Component: base Version: 1.7.0
Keywords: Cc: raimue (Rainer Müller)
Port:

Description

Why does the user care that a port is being built "with target all"?

$ port install foo
--->  Fetching foo
--->  Verifying checksum(s) for foo
--->  Extracting foo
--->  Applying patches to foo
--->  Configuring foo
--->  Building foo with target all
--->  Staging foo into destroot
--->  Installing foo @1.2.3_0
--->  Activating foo @1.2.3_0
--->  Cleaning foo
$

The user is not informed of any other minutiae of the build process, for example whether the source was extracted from a bzip2 or a gzip or a zip archive, or whether a particular configure argument was needed, or whether the destroot phase needed to inform the install process of the destroot path via different mechanism. The user doesn't care about all that, nor does he care that the "make" command happened to want the argument "all" passed to it. And if he does, he can use the verbose or debug switches. So can we please remove "with target <whatever>" from the "Building" info line?

Attachments (1)

patch-portbuild.tcl.diff (617 bytes) - added by raimue (Rainer Müller) 16 years ago.

Download all attachments as: .zip

Change History (6)

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

Component: portsbase

Changed 16 years ago by raimue (Rainer Müller)

Attachment: patch-portbuild.tcl.diff added

comment:2 Changed 16 years ago by raimue (Rainer Müller)

I agree that this is unnecessary information for the end user.

Attached a patch to remove it.

comment:3 Changed 16 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: fixed
Status: newclosed

Thanks Rainer. That looks good. I committed it in r38315.

Goodness, that had been with us since r221.

comment:4 Changed 16 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: raimue@… added

Noted in ChangeLog in r38316.

comment:5 Changed 16 years ago by blb@…

Milestone: MacPorts base bugsMacPorts 1.7.0
Note: See TracTickets for help on using tickets.