1
0
Fork 0
mirror of https://git.tukaani.org/xz.git synced 2024-04-04 12:36:23 +02:00

xz: Make xz buildable even when encoders or decoders are disabled.

The patch is quite long but it's mostly about adding new #ifdefs
to omit code when encoders or decoders have been disabled.

This adds two new #defines to config.h: HAVE_ENCODERS and
HAVE_DECODERS.
This commit is contained in:
Lasse Collin 2015-11-03 20:29:33 +02:00
parent 4cc584985c
commit cb3111e3ed
6 changed files with 62 additions and 13 deletions

View file

@ -114,6 +114,8 @@ else
;; ;;
esac esac
done done
AC_DEFINE([HAVE_ENCODERS], [1],
[Define to 1 if any of HAVE_ENCODER_foo have been defined.])
AC_MSG_RESULT([$enable_encoders]) AC_MSG_RESULT([$enable_encoders])
fi fi
@ -141,6 +143,8 @@ else
;; ;;
esac esac
done done
AC_DEFINE([HAVE_DECODERS], [1],
[Define to 1 if any of HAVE_DECODER_foo have been defined.])
AC_MSG_RESULT([$enable_decoders]) AC_MSG_RESULT([$enable_decoders])
fi fi

View file

@ -16,8 +16,6 @@ xz_SOURCES = \
file_io.h \ file_io.h \
hardware.c \ hardware.c \
hardware.h \ hardware.h \
list.c \
list.h \
main.c \ main.c \
main.h \ main.h \
message.c \ message.c \
@ -39,6 +37,12 @@ xz_SOURCES = \
../common/tuklib_mbstr_width.c \ ../common/tuklib_mbstr_width.c \
../common/tuklib_mbstr_fw.c ../common/tuklib_mbstr_fw.c
if COND_MAIN_DECODER
xz_SOURCES = \
list.c \
list.h
endif
if COND_W32 if COND_W32
xz_SOURCES += xz_w32res.rc xz_SOURCES += xz_w32res.rc
endif endif

View file

@ -635,6 +635,22 @@ args_parse(args_info *args, int argc, char **argv)
// Then from the command line // Then from the command line
parse_real(args, argc, argv); parse_real(args, argc, argv);
// If encoder or decoder support was omitted at build time,
// show an error now so that the rest of the code can rely on
// that whatever is in opt_mode is also supported.
#ifndef HAVE_ENCODERS
if (opt_mode == MODE_COMPRESS)
message_fatal(_("Compression support was disabled "
"at build time"));
#endif
#ifndef HAVE_DECODERS
// Even MODE_LIST cannot work without decoder support so MODE_COMPRESS
// is the only valid choice.
if (opt_mode != MODE_COMPRESS)
message_fatal(_("Decompression support was disabled "
"at build time"));
#endif
// Never remove the source file when the destination is not on disk. // Never remove the source file when the destination is not on disk.
// In test mode the data is written nowhere, but setting opt_stdout // In test mode the data is written nowhere, but setting opt_stdout
// will make the rest of the code behave well. // will make the rest of the code behave well.

View file

@ -51,7 +51,7 @@ static lzma_check check;
/// This becomes false if the --check=CHECK option is used. /// This becomes false if the --check=CHECK option is used.
static bool check_default = true; static bool check_default = true;
#ifdef MYTHREAD_ENABLED #if defined(HAVE_ENCODERS) && defined(MYTHREAD_ENABLED)
static lzma_mt mt_options = { static lzma_mt mt_options = {
.flags = 0, .flags = 0,
.timeout = 300, .timeout = 300,
@ -221,8 +221,9 @@ coder_set_compression_settings(void)
// Get the memory usage. Note that if --format=raw was used, // Get the memory usage. Note that if --format=raw was used,
// we can be decompressing. // we can be decompressing.
const uint64_t memory_limit = hardware_memlimit_get(opt_mode); const uint64_t memory_limit = hardware_memlimit_get(opt_mode);
uint64_t memory_usage; uint64_t memory_usage = UINT64_MAX;
if (opt_mode == MODE_COMPRESS) { if (opt_mode == MODE_COMPRESS) {
#ifdef HAVE_ENCODERS
# ifdef MYTHREAD_ENABLED # ifdef MYTHREAD_ENABLED
if (opt_format == FORMAT_XZ && hardware_threads_get() > 1) { if (opt_format == FORMAT_XZ && hardware_threads_get() > 1) {
mt_options.threads = hardware_threads_get(); mt_options.threads = hardware_threads_get();
@ -239,8 +240,11 @@ coder_set_compression_settings(void)
{ {
memory_usage = lzma_raw_encoder_memusage(filters); memory_usage = lzma_raw_encoder_memusage(filters);
} }
#endif
} else { } else {
#ifdef HAVE_DECODERS
memory_usage = lzma_raw_decoder_memusage(filters); memory_usage = lzma_raw_decoder_memusage(filters);
#endif
} }
if (memory_usage == UINT64_MAX) if (memory_usage == UINT64_MAX)
@ -248,7 +252,11 @@ coder_set_compression_settings(void)
// Print memory usage info before possible dictionary // Print memory usage info before possible dictionary
// size auto-adjusting. // size auto-adjusting.
//
// NOTE: If only encoder support was built, we cannot show the
// what the decoder memory usage will be.
message_mem_needed(V_DEBUG, memory_usage); message_mem_needed(V_DEBUG, memory_usage);
#ifdef HAVE_DECODERS
if (opt_mode == MODE_COMPRESS) { if (opt_mode == MODE_COMPRESS) {
const uint64_t decmem = lzma_raw_decoder_memusage(filters); const uint64_t decmem = lzma_raw_decoder_memusage(filters);
if (decmem != UINT64_MAX) if (decmem != UINT64_MAX)
@ -256,6 +264,7 @@ coder_set_compression_settings(void)
"%s MiB of memory."), uint64_to_str( "%s MiB of memory."), uint64_to_str(
round_up_to_mib(decmem), 0)); round_up_to_mib(decmem), 0));
} }
#endif
if (memory_usage <= memory_limit) if (memory_usage <= memory_limit)
return; return;
@ -268,6 +277,7 @@ coder_set_compression_settings(void)
assert(opt_mode == MODE_COMPRESS); assert(opt_mode == MODE_COMPRESS);
#ifdef HAVE_ENCODERS
# ifdef MYTHREAD_ENABLED # ifdef MYTHREAD_ENABLED
if (opt_format == FORMAT_XZ && mt_options.threads > 1) { if (opt_format == FORMAT_XZ && mt_options.threads > 1) {
// Try to reduce the number of threads before // Try to reduce the number of threads before
@ -349,11 +359,13 @@ coder_set_compression_settings(void)
uint64_to_str(orig_dict_size >> 20, 0), uint64_to_str(orig_dict_size >> 20, 0),
uint64_to_str(opt->dict_size >> 20, 1), uint64_to_str(opt->dict_size >> 20, 1),
uint64_to_str(round_up_to_mib(memory_limit), 2)); uint64_to_str(round_up_to_mib(memory_limit), 2));
#endif
return; return;
} }
#ifdef HAVE_DECODERS
/// Return true if the data in in_buf seems to be in the .xz format. /// Return true if the data in in_buf seems to be in the .xz format.
static bool static bool
is_format_xz(void) is_format_xz(void)
@ -411,6 +423,7 @@ is_format_lzma(void)
return true; return true;
} }
#endif
/// Detect the input file type (for now, this done only when decompressing), /// Detect the input file type (for now, this done only when decompressing),
@ -424,6 +437,7 @@ coder_init(file_pair *pair)
lzma_ret ret = LZMA_PROG_ERROR; lzma_ret ret = LZMA_PROG_ERROR;
if (opt_mode == MODE_COMPRESS) { if (opt_mode == MODE_COMPRESS) {
#ifdef HAVE_ENCODERS
switch (opt_format) { switch (opt_format) {
case FORMAT_AUTO: case FORMAT_AUTO:
// args.c ensures this. // args.c ensures this.
@ -449,7 +463,9 @@ coder_init(file_pair *pair)
ret = lzma_raw_encoder(&strm, filters); ret = lzma_raw_encoder(&strm, filters);
break; break;
} }
#endif
} else { } else {
#ifdef HAVE_DECODERS
uint32_t flags = 0; uint32_t flags = 0;
// It seems silly to warn about unsupported check if the // It seems silly to warn about unsupported check if the
@ -531,6 +547,7 @@ coder_init(file_pair *pair)
strm.avail_out = 0; strm.avail_out = 0;
ret = lzma_code(&strm, LZMA_RUN); ret = lzma_code(&strm, LZMA_RUN);
} }
#endif
} }
if (ret != LZMA_OK) { if (ret != LZMA_OK) {

View file

@ -225,8 +225,11 @@ main(int argc, char **argv)
// coder_run() handles compression, decompression, and testing. // coder_run() handles compression, decompression, and testing.
// list_file() is for --list. // list_file() is for --list.
void (*run)(const char *filename) = opt_mode == MODE_LIST void (*run)(const char *filename) = &coder_run;
? &list_file : &coder_run; #ifdef HAVE_DECODERS
if (opt_mode == MODE_LIST)
run = &list_file;
#endif
// Process the files given on the command line. Note that if no names // Process the files given on the command line. Note that if no names
// were given, args_parse() gave us a fake "-" filename. // were given, args_parse() gave us a fake "-" filename.
@ -285,6 +288,7 @@ main(int argc, char **argv)
(void)fclose(args.files_file); (void)fclose(args.files_file);
} }
#ifdef HAVE_DECODERS
// All files have now been handled. If in --list mode, display // All files have now been handled. If in --list mode, display
// the totals before exiting. We don't have signal handlers // the totals before exiting. We don't have signal handlers
// enabled in --list mode, so we don't need to check user_abort. // enabled in --list mode, so we don't need to check user_abort.
@ -292,6 +296,7 @@ main(int argc, char **argv)
assert(!user_abort); assert(!user_abort);
list_totals(); list_totals();
} }
#endif
#ifndef NDEBUG #ifndef NDEBUG
coder_free(); coder_free();

View file

@ -60,4 +60,7 @@
#include "signals.h" #include "signals.h"
#include "suffix.h" #include "suffix.h"
#include "util.h" #include "util.h"
#ifdef HAVE_DECODERS
# include "list.h" # include "list.h"
#endif