library: remove procps_version functions

It doesn't make any sense to have the binary version strings
embedded into the library. The version strings are defined
already either in the Makefile or in include/c.h
This commit is contained in:
Craig Small 2015-06-18 22:37:24 +10:00
parent 6577371919
commit 505f257a8c
8 changed files with 15 additions and 21 deletions

View File

@ -53,7 +53,6 @@
#include "proc/sig.h" #include "proc/sig.h"
#include "proc/devname.h" #include "proc/devname.h"
#include "proc/sysinfo.h" #include "proc/sysinfo.h"
#include "proc/version.h" /* procps_version */
static int i_am_pkill = 0; static int i_am_pkill = 0;

View File

@ -4,7 +4,6 @@ global:
closeproc; closeproc;
cpuinfo; cpuinfo;
dev_to_tty; dev_to_tty;
display_version;
escape_command; escape_command;
escape_str; escape_str;
escape_strlist; escape_strlist;
@ -47,7 +46,6 @@ global:
page_bytes; page_bytes;
pretty_print_signals; pretty_print_signals;
print_uptime; print_uptime;
procps_version;
put_slabinfo; put_slabinfo;
readeither; readeither;
readproc; readproc;

View File

@ -24,12 +24,6 @@
#include <stdlib.h> #include <stdlib.h>
#include "version.h" #include "version.h"
const char procps_version[] = PACKAGE_NAME " version " PACKAGE_VERSION;
void display_version(void) {
fprintf(stdout, "%s\n", procps_version);
}
/* Linux kernel version information for procps-ng utilities /* Linux kernel version information for procps-ng utilities
* Copyright (c) 1996 Charles Blake <cblake@bbn.com> * Copyright (c) 1996 Charles Blake <cblake@bbn.com>
*/ */

View File

@ -15,8 +15,6 @@
EXTERN_C_BEGIN EXTERN_C_BEGIN
void init_Linux_version(void); /* Get Linux version */ void init_Linux_version(void); /* Get Linux version */
extern void display_version(void); /* display suite version */
extern const char procps_version[]; /* global buf for suite version */
extern int linux_version_code; /* runtime version of LINUX_VERSION_CODE extern int linux_version_code; /* runtime version of LINUX_VERSION_CODE
in /usr/include/linux/version.h */ in /usr/include/linux/version.h */

View File

@ -55,7 +55,7 @@ static void signal_handler(int signo){
signo, signo,
signal_number_to_name(signo), signal_number_to_name(signo),
myname, myname,
procps_version PACKAGE_VERSION
); );
switch (signo) { switch (signo) {
case SIGHUP: case SIGHUP:

View File

@ -479,7 +479,7 @@ void self_info(void){
sysv_l_format ? sysv_l_format : "(none)" sysv_l_format ? sysv_l_format : "(none)"
); );
display_version(); fprintf(stderr, "%s version %s\n", PACKAGE_NAME, PACKAGE_VERSION);
fprintf(stderr, "Linux version %d.%d.%d\n", fprintf(stderr, "Linux version %d.%d.%d\n",
LINUX_VERSION_MAJOR(linux_version_code), LINUX_VERSION_MAJOR(linux_version_code),
LINUX_VERSION_MINOR(linux_version_code), LINUX_VERSION_MINOR(linux_version_code),

View File

@ -32,9 +32,9 @@
#include <sys/types.h> #include <sys/types.h>
#include "../proc/alloc.h" #include "../proc/alloc.h"
#include "../proc/version.h"
#include "common.h" #include "common.h"
#include "c.h"
#define ARG_GNU 0 #define ARG_GNU 0
#define ARG_END 1 #define ARG_END 1
@ -58,6 +58,11 @@ static int force_bsd = 0; /* set when normal parsing fails */
return _("the option is exclusive: " x) return _("the option is exclusive: " x)
/********** utility functions **********/ /********** utility functions **********/
static void display_ps_version(void)
{
fprintf(stdout, PROCPS_NG_VERSION);
}
/* /*
* Both "-Oppid" and "-O ppid" should be legal, though Unix98 * Both "-Oppid" and "-O ppid" should be legal, though Unix98
@ -341,7 +346,7 @@ static const char *parse_sysv_option(void){
case 'V': /* single */ case 'V': /* single */
trace("-V prints version\n"); trace("-V prints version\n");
exclusive("-V"); exclusive("-V");
display_version(); display_ps_version();
exit(0); exit(0);
// This must be verified against SVR4-MP. (UnixWare or Powermax) // This must be verified against SVR4-MP. (UnixWare or Powermax)
// Leave it undocumented until that problem is solved. // Leave it undocumented until that problem is solved.
@ -610,7 +615,7 @@ static const char *parse_bsd_option(void){
case 'V': /* single */ case 'V': /* single */
trace("V show version info\n"); trace("V show version info\n");
exclusive("V"); exclusive("V");
display_version(); display_ps_version();
exit(0); exit(0);
case 'W': case 'W':
trace("W N/A get swap info from ... not /dev/drum.\n"); trace("W N/A get swap info from ... not /dev/drum.\n");
@ -1029,7 +1034,7 @@ static const char *parse_gnu_option(void){
case_version: case_version:
trace("--version\n"); trace("--version\n");
exclusive("--version"); exclusive("--version");
display_version(); display_ps_version();
exit(0); exit(0);
return NULL; return NULL;
case_context: case_context:

View File

@ -445,7 +445,7 @@ static void bye_bye (const char *str) {
, LINUX_VERSION_MAJOR(linux_version_code) , LINUX_VERSION_MAJOR(linux_version_code)
, LINUX_VERSION_MINOR(linux_version_code) , LINUX_VERSION_MINOR(linux_version_code)
, LINUX_VERSION_PATCH(linux_version_code) , LINUX_VERSION_PATCH(linux_version_code)
, procps_version , PACKAGE_STRING
, (unsigned)Hertz, (unsigned)sizeof(Hertz), (unsigned)sizeof(Hertz) * 8 , (unsigned)Hertz, (unsigned)sizeof(Hertz), (unsigned)sizeof(Hertz) * 8
, (int)page_bytes, Cpu_faux_tot, (int)smp_num_cpus, (unsigned)sizeof(CPU_t) , (int)page_bytes, Cpu_faux_tot, (int)smp_num_cpus, (unsigned)sizeof(CPU_t)
, (unsigned)sizeof(HST_t), ((int)page_bytes / (int)sizeof(HST_t)), HHist_siz , (unsigned)sizeof(HST_t), ((int)page_bytes / (int)sizeof(HST_t)), HHist_siz
@ -3696,7 +3696,7 @@ static void parse_args (char **args) {
case 'h': case 'h':
case 'v': case 'v':
puts(fmtmk(N_fmt(HELP_cmdline_fmt) puts(fmtmk(N_fmt(HELP_cmdline_fmt)
, procps_version, Myname, N_txt(USAGE_abbrev_txt))); , PACKAGE_STRING, Myname, N_txt(USAGE_abbrev_txt)));
bye_bye(NULL); bye_bye(NULL);
case 'i': case 'i':
TOGw(Curwin, Show_IDLEPS); TOGw(Curwin, Show_IDLEPS);
@ -3975,7 +3975,7 @@ signify_that:
putp(Cap_home); putp(Cap_home);
// this string is well above ISO C89's minimum requirements! // this string is well above ISO C89's minimum requirements!
show_special(1, fmtmk(N_unq(COLOR_custom_fmt) show_special(1, fmtmk(N_unq(COLOR_custom_fmt)
, procps_version, w->grpname , PACKAGE_STRING, w->grpname
, CHKw(w, View_NOBOLD) ? N_txt(ON_word_only_txt) : N_txt(OFF_one_word_txt) , CHKw(w, View_NOBOLD) ? N_txt(ON_word_only_txt) : N_txt(OFF_one_word_txt)
, CHKw(w, Show_COLORS) ? N_txt(ON_word_only_txt) : N_txt(OFF_one_word_txt) , CHKw(w, Show_COLORS) ? N_txt(ON_word_only_txt) : N_txt(OFF_one_word_txt)
, CHKw(w, Show_HIBOLD) ? N_txt(ON_word_only_txt) : N_txt(OFF_one_word_txt) , CHKw(w, Show_HIBOLD) ? N_txt(ON_word_only_txt) : N_txt(OFF_one_word_txt)
@ -4219,7 +4219,7 @@ signify_that:
adj_geometry(); adj_geometry();
show_special(1, fmtmk(N_unq(KEYS_helpbas_fmt) show_special(1, fmtmk(N_unq(KEYS_helpbas_fmt)
, procps_version , PACKAGE_STRING
, w->grpname , w->grpname
, CHKw(w, Show_CTIMES) ? N_txt(ON_word_only_txt) : N_txt(OFF_one_word_txt) , CHKw(w, Show_CTIMES) ? N_txt(ON_word_only_txt) : N_txt(OFF_one_word_txt)
, Rc.delay_time , Rc.delay_time