After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 653637 - gnome-nettool should use 'finger' instead of 'pinky' if finger exists.
gnome-nettool should use 'finger' instead of 'pinky' if finger exists.
Status: RESOLVED FIXED
Product: gnome-nettool
Classification: Applications
Component: general
git master
Other All
: Normal major
: ---
Assigned To: Germán Poo-Caamaño
Rodrigo Moya
: 611408 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2011-06-29 11:35 UTC by ayan.george
Modified: 2011-07-13 07:07 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to add preference for 'finger' instead of 'pinky' in gnome-nettool (639 bytes, patch)
2011-06-29 11:35 UTC, ayan.george
accepted-commit_now Details | Review

Description ayan.george 2011-06-29 11:35:18 UTC
Created attachment 190923 [details] [review]
Patch to add preference for 'finger' instead of 'pinky' in gnome-nettool

Currently, gnome-nettool will use 'pinky' even if 'finger' exists on a system.

The code for selecting a finger client is as follows:

  program = util_find_program_in_path ("pinky", NULL);

  if (program != NULL) {
    program_name = g_strdup ("pinky");
  } else {
    program = util_find_program_dialog ("finger", parent);
    program_name = g_strdup ("finger");
  }

This is a problem because:

* pinky is by definition a light-weight replacement for finger that does not actually use the finger network protocol -- it only examines the local utmp file.
* Even if both full-featured finger and pinky are installed, gnome-nettool still uses pinky.
* Many non-Linux systems (like *BSD) have proper finger programs and it is odd that gnome-nettool prefers a non-standard client.

The code should first attempt to find a finger program and fall back to pinky if it exists -- not the other way around.
Comment 1 Rodrigo Moya 2011-07-12 08:39:51 UTC
pinky is being used because of this: https://bugzilla.gnome.org/show_bug.cgi?id=549588
Comment 2 ayan.george 2011-07-12 15:18:09 UTC
(In reply to comment #1)
> pinky is being used because of this:
> https://bugzilla.gnome.org/show_bug.cgi?id=549588

Right -- but implementation is broken.  The code should first look for 'finger' then look for 'pinky' -- not the other way around.
Comment 3 Germán Poo-Caamaño 2011-07-12 15:24:29 UTC
Review of attachment 190923 [details] [review]:

The patch looks ok to me. Please go ahead and commit it.
Comment 4 Germán Poo-Caamaño 2011-07-13 03:19:20 UTC
This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.
Comment 5 Germán Poo-Caamaño 2011-07-13 07:06:26 UTC
This also has been fixed in gnome-2-32.
Comment 6 Germán Poo-Caamaño 2011-07-13 07:07:18 UTC
*** Bug 611408 has been marked as a duplicate of this bug. ***