Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 197990 - etc-update does not show any diffs: missing "dialog"...
Summary: etc-update does not show any diffs: missing "dialog"...
Status: RESOLVED FIXED
Alias: None
Product: Portage Development
Classification: Unclassified
Component: Core (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Portage team
URL:
Whiteboard:
Keywords: InVCS
Depends on:
Blocks: 198097
  Show dependency tree
 
Reported: 2007-11-03 19:11 UTC by Mark Dominik Bürkle
Modified: 2007-11-04 22:14 UTC (History)
0 users

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


Attachments
check for functionality of "dialog" w/switch to text-mode (etc-update.diff,547 bytes, patch)
2007-11-03 19:49 UTC, Mark Dominik Bürkle
Details | Diff
check dialog functionality on startup and on every use. (etc-update.diff,2.79 KB, patch)
2007-11-03 20:23 UTC, Mark Dominik Bürkle
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mark Dominik Bürkle 2007-11-03 19:11:57 UTC
In my fresh 2007.0 amd64 install there is no "dialog" installed.
Just modifying the etc-update.conf (mode=1) leads to permanent error

  Exiting: User termination!

The reason for this is that there is no errorcode checking after the call of "dialog --menu ...", only a minimalistic "exit-on-error" is provided.

The patch following this post is introducing a simple check for this:

  [[ -z $( type -p dialog ) ]] && echo "etc-update: WARNING -- dialog not found, using non-menu mode." && mode=0



The patch http://bugs.gentoo.org/attachment.cgi?id=13668 from Bug#23243 introduces another possibility to catch this. I also introduced "auto merge all" and "discard all" in the dialog execution branches of etc-update.

The last change in Bug#23243 in attachment#13668 [details, diff] introduces the viewing of the diff-output within "dialog", including choices. I personally prefer vimdiff, so I did not check the remaining code (check max display size)...


Kind regards,
Dominik
Comment 1 Mark Dominik Bürkle 2007-11-03 19:49:20 UTC
Created attachment 135113 [details, diff]
check for functionality of "dialog" w/switch to text-mode
Comment 2 Mark Dominik Bürkle 2007-11-03 20:23:37 UTC
Created attachment 135121 [details, diff]
check dialog functionality on startup and on every use.

Also includes choices for "auto merge all" and "remove all updates" as in the text version.
Does NOT include modification to show diff-output in a dialog window as proposed in previous bug report (see above).
Comment 3 SpanKY gentoo-dev 2007-11-03 21:13:45 UTC
checking dialog at every step is just silly

ive added a check at startup similar to your first patch, thanks
Comment 4 Zac Medico gentoo-dev 2007-11-04 22:14:19 UTC
This has been released in 2.1.3.18.