128373 – swapping --progress for --verbose with rsync options

Bug 128373 - swapping --progress for --verbose with rsync options

Summary: swapping --progress for --verbose with rsync options

Status:	RESOLVED FIXED

Alias:	None

Product:	Portage Development
Classification:	Unclassified
Component:	Core (show other bugs)
Hardware:	All Linux

Importance:	High normal (vote)
Assignee:	Portage team

URL:
Whiteboard:
Keywords:	InVCS

Depends on:
Blocks:	115839
	Show dependency tree

Reported:	2006-04-01 05:20 UTC by Olaf Hering
Modified:	2006-04-09 00:16 UTC (History)
CC List:	1 user (show)

See Also:
Package list:
Runtime testing required:	---

Attachments
portage.progress.patch (portage.progress.patch,658 bytes, patch) 2006-04-01 05:21 UTC, Olaf Hering	Details \| Diff
portage.progress.patch (portage.progress.patch,709 bytes, patch) 2006-04-01 09:53 UTC, Olaf Hering	Details \| Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Olaf Hering 2006-04-01 05:20:51 UTC

there is not much point for rsync -P. It clutters the logs.
Those who have slow links and care about the transferrates can give --verbose somewhere.

Comment 1 Olaf Hering 2006-04-01 05:21:19 UTC

Created attachment 83610 [details, diff]
portage.progress.patch

Comment 2 Radek Podgorny 2006-04-01 06:45:19 UTC

Well, I really like seeing some progress so I'm against the removal but I agree it should be switchable (for example being off the you "emerge sync" and on when you "emerge sync -v").

BTW, there's already some tweaking possibe in /etc/make.conf (see /etc/make.conf.example)...

Comment 3 Radek Podgorny 2006-04-01 06:46:22 UTC

Ha, now I've seen your patch. Looks good to me...

Comment 4 Olaf Hering 2006-04-01 09:53:39 UTC

Created attachment 83626 [details, diff]
portage.progress.patch

-P implies -v. 
So, just -v per default, and -P when --verbose is passed to emerge.

this is what I meant to change.

Comment 5 Simon Stelling (RETIRED) gentoo-dev

2006-04-06 14:05:17 UTC

Finally -v gets a meaning ;)
Thanks for the patch, this is rev 3084

Comment 6 Zac Medico gentoo-dev

2006-04-09 00:16:54 UTC

Released in 2.1_pre7-r5.