mirror of
https://github.com/moparisthebest/pacman
synced 2024-12-21 23:38:49 -05:00
merge _alpm_logaction into alpm_logaction
Signed-off-by: Andrew Gregory <andrew.gregory.8@gmail.com> Signed-off-by: Allan McRae <allan@archlinux.org>
This commit is contained in:
parent
f9423cfa5d
commit
ae7067440c
@ -21,6 +21,7 @@
|
||||
#include <stdio.h>
|
||||
#include <stdarg.h>
|
||||
#include <errno.h>
|
||||
#include <syslog.h>
|
||||
|
||||
/* libalpm */
|
||||
#include "log.h"
|
||||
@ -42,11 +43,15 @@
|
||||
int SYMEXPORT alpm_logaction(alpm_handle_t *handle, const char *prefix,
|
||||
const char *fmt, ...)
|
||||
{
|
||||
int ret;
|
||||
int ret = 0;
|
||||
va_list args;
|
||||
|
||||
ASSERT(handle != NULL, return -1);
|
||||
|
||||
if(!(prefix && *prefix)) {
|
||||
prefix = "UNKNOWN";
|
||||
}
|
||||
|
||||
/* check if the logstream is open already, opening it if needed */
|
||||
if(handle->logstream == NULL) {
|
||||
int fd;
|
||||
@ -54,11 +59,8 @@ int SYMEXPORT alpm_logaction(alpm_handle_t *handle, const char *prefix,
|
||||
fd = open(handle->logfile, O_WRONLY | O_APPEND | O_CREAT | O_CLOEXEC,
|
||||
0644);
|
||||
} while(fd == -1 && errno == EINTR);
|
||||
if(fd >= 0) {
|
||||
handle->logstream = fdopen(fd, "a");
|
||||
}
|
||||
/* if we couldn't open it, we have an issue */
|
||||
if(fd < 0 || handle->logstream == NULL) {
|
||||
if(fd < 0 || (handle->logstream = fdopen(fd, "a")) == NULL) {
|
||||
if(errno == EACCES) {
|
||||
handle->pm_errno = ALPM_ERR_BADPERMS;
|
||||
} else if(errno == ENOENT) {
|
||||
@ -71,9 +73,29 @@ int SYMEXPORT alpm_logaction(alpm_handle_t *handle, const char *prefix,
|
||||
}
|
||||
|
||||
va_start(args, fmt);
|
||||
ret = _alpm_logaction(handle, prefix, fmt, args);
|
||||
va_end(args);
|
||||
|
||||
if(handle->usesyslog) {
|
||||
/* we can't use a va_list more than once, so we need to copy it
|
||||
* so we can use the original when calling vfprintf below. */
|
||||
va_list args_syslog;
|
||||
va_copy(args_syslog, args);
|
||||
vsyslog(LOG_WARNING, fmt, args_syslog);
|
||||
va_end(args_syslog);
|
||||
}
|
||||
|
||||
if(handle->logstream) {
|
||||
time_t t = time(NULL);
|
||||
struct tm *tm = localtime(&t);
|
||||
|
||||
/* Use ISO-8601 date format */
|
||||
fprintf(handle->logstream, "[%04d-%02d-%02d %02d:%02d] [%s] ",
|
||||
tm->tm_year + 1900, tm->tm_mon + 1, tm->tm_mday,
|
||||
tm->tm_hour, tm->tm_min, prefix);
|
||||
ret = vfprintf(handle->logstream, fmt, args);
|
||||
fflush(handle->logstream);
|
||||
}
|
||||
|
||||
va_end(args);
|
||||
return ret;
|
||||
}
|
||||
|
||||
|
@ -27,7 +27,6 @@
|
||||
#include <ctype.h>
|
||||
#include <dirent.h>
|
||||
#include <time.h>
|
||||
#include <syslog.h>
|
||||
#include <errno.h>
|
||||
#include <limits.h>
|
||||
#include <sys/wait.h>
|
||||
@ -446,49 +445,6 @@ ssize_t _alpm_files_in_directory(alpm_handle_t *handle, const char *path,
|
||||
return files;
|
||||
}
|
||||
|
||||
/** Write formatted message to log.
|
||||
* @param handle the context handle
|
||||
* @param format formatted string to write out
|
||||
* @param args formatting arguments
|
||||
* @return 0 or number of characters written on success, vfprintf return value
|
||||
* on error
|
||||
*/
|
||||
int _alpm_logaction(alpm_handle_t *handle, const char *prefix,
|
||||
const char *fmt, va_list args)
|
||||
{
|
||||
int ret = 0;
|
||||
|
||||
if(!(prefix && *prefix)) {
|
||||
prefix = "UNKNOWN";
|
||||
}
|
||||
|
||||
if(handle->usesyslog) {
|
||||
/* we can't use a va_list more than once, so we need to copy it
|
||||
* so we can use the original when calling vfprintf below. */
|
||||
va_list args_syslog;
|
||||
va_copy(args_syslog, args);
|
||||
vsyslog(LOG_WARNING, fmt, args_syslog);
|
||||
va_end(args_syslog);
|
||||
}
|
||||
|
||||
if(handle->logstream) {
|
||||
time_t t;
|
||||
struct tm *tm;
|
||||
|
||||
t = time(NULL);
|
||||
tm = localtime(&t);
|
||||
|
||||
/* Use ISO-8601 date format */
|
||||
fprintf(handle->logstream, "[%04d-%02d-%02d %02d:%02d] [%s] ",
|
||||
tm->tm_year + 1900, tm->tm_mon + 1, tm->tm_mday,
|
||||
tm->tm_hour, tm->tm_min, prefix);
|
||||
ret = vfprintf(handle->logstream, fmt, args);
|
||||
fflush(handle->logstream);
|
||||
}
|
||||
|
||||
return ret;
|
||||
}
|
||||
|
||||
/** Execute a command with arguments in a chroot.
|
||||
* @param handle the context handle
|
||||
* @param cmd command to execute
|
||||
|
@ -119,9 +119,6 @@ int _alpm_unpack(alpm_handle_t *handle, const char *archive, const char *prefix,
|
||||
|
||||
ssize_t _alpm_files_in_directory(alpm_handle_t *handle, const char *path, int full_count);
|
||||
|
||||
int _alpm_logaction(alpm_handle_t *handle, const char *prefix, const char *fmt, va_list args)
|
||||
__attribute__((format(printf, 3, 0)));
|
||||
|
||||
int _alpm_run_chroot(alpm_handle_t *handle, const char *cmd, char *const argv[]);
|
||||
int _alpm_ldconfig(alpm_handle_t *handle);
|
||||
int _alpm_str_cmp(const void *s1, const void *s2);
|
||||
|
Loading…
Reference in New Issue
Block a user