GNOME Bugzilla – Bug 129176
[PATCH] Fix some FreeBSD issues in gnome-system-monitor
Last modified: 2011-11-11 10:03:55 UTC
The attached patch fixes some compilation issues and unresolved symbol issues in gnome-system-monitor-2.5.2. There may be some functionality lost due to the lack of grantpt and unlockpt.
Created attachment 22363 [details] [review] Fix FreeBSD issues in gnome-system-monitor
Looks good to me. Can you commit it yourself? Thanks a lot :)
Nope, sorry. I'm just a lowly porter :-).
Adding the portability keyword.
The original patch in this report is quite limited, because it only recognizes FreeBSD and assumes that functions are not there based on the fact it's FreeBSD (this is right, but what will happen if, some day, these functions become part of it?). gnome-system-monitor also has build problems in NetBSD, because all these functions are missing, too. Furthermore, the program needs to be linked against -lutil, which provides ttyname(3). I've attached a new patch that aims to be more flexible. It checks for these functions from configure and also adds -lutil to LIBS in case we are running NetBSD. It could be very good if this was applied before GNOME 2.6 ;-)
Created attachment 24549 [details] [review] Patch to fix FreeBSD and support NetBSD
Yes, this is much better.
Looks like Kevin accepted the first patch... since people seems to agree that the second one is better, can this go in now that we are post freeze?
there's nothing left to fix as procman now uses libgnomesu