Avoid spurious test failures due to buggy ncurses-5.6.
* tests/test-lib.sh: Also unset TERM.

partprobe: new option: --dry-run, synonym for now-deprecated --no-update; improve its description

Test commit

Removed clean.sh script. Using git now, we don't need it anymore (git clean -d -x).

Use gnulib xmalloc() and xrealloc().

Fix exception handling in mkpart and mkpartfs

Parted mkpart and mkpartfs commands does:
...
   ped_exception_fetch_all();
   if (!ped_disk_add_partition (disk, part, final_constraint)) {
 ped_exception_leave_all();
...
   }
...
In mkpart if the ped_disk_add_partition() returns true it skips
ped_exception_leave_all() leaving ex_fetch_count = 1.

In mkpartfs if the ped_disk_add_partition() returns false it will
call ped_exception_leave_all() leaving ex_fetch_count negative.

The wrong count in ex_fetch_count will prevent next commands to
correctly handle exceptions, failing to prompt users for example.

Note: test script t2000-mkfs.sh improved to verify against this issue.

Signed-off-by: Flavio Leitner <flavio.leitner@gmail.com>
Signed-off-by: Jim Meyering <jim@meyering.net>

* tests/t1100-busy-label.sh: Quote uses of $dev, in case
the user's device name contains a shell meta-character.

SCALAR(0x7f6040)

When labeling a disk in --script mode, fail if it is in use.
* parted/parted.c (_disk_warn_busy): In script mode,
throw a "PED_EXCEPTION_ERROR", not a warning.
(do_mklabel): Guard only the _disk_warn_loss call with
"if (!opt_script_mode...", not the _disk_warn_loss call.
* tests/t1100-busy-label.sh: New file.  Test the above,
in interactive mode as well as in script mode.  Requires
root privilege (to mount a fs), and an actual block device.
* tests/Makefile.am (TESTS): Add t1100-busy-label.sh.
* tests/test-lib.sh: Add infrastructure to support new
privileges_required_=1 and erasable_device_required_=1
settings used by t1100.

No longer require "srcdir=." when running a test manually.
* tests/Makefile.am (init.sh): New rule, so one doesn't have to
set "srcdir=." when running a test manually.
* .gitignore: Ignore new generated file: tests/init.sh.
* tests/t0000-basic.sh: Source new init.sh, rather than test-lib.sh.
* tests/t1000-mkpartfs.sh: Likewise.
* tests/t2000-mkfs.sh: Likewise.
* tests/test-lib.sh: Remove useless srcdir-setting code.

[parted] Fix script mode support on mkfs commandline command

Parted was lacking support to script mode on do_mkfs method hence
always failing.

Note: tests/t2000-mkfs.sh was created to avoid it to happen again

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

Preventing compilation of DASD code on GNU Hurd systems through the use of AC_COMPILE_IFELSE and AM_CONDITIONAL, instead of #ifdef.

Rewrite integration tests to use a new framework (git's).
* tests/t1000-mkpartfs.sh: New file.
* tests/t0000-basic.sh: New file.
* tests/test-lib.sh: New file.  Derived from git's t/test-lib.sh.
* tests/lang-default, tests/priv-check: Remove files.
* tests/Makefile.am (EXTRA_DIST): Add test-lib.sh.
Remove lang-default and priv-check; no longer used.
* tests/part-01, tests/label-01: Remove files.
* tests/part-02, tests/label-02: Likewise.

Merge branch 'inhibit-word-wrap'

Merge branch 'include-limits-h-for-CHAR_MAX'

Make ---pretend-input-tty inhibit output word-wrap.
* parted/ui.c (screen_width): Use effectively-unlimited screen width
with ---pretend-input-tty, just as for '--script' mode.

Suppress "you are not superuser..." warning in script mode.
* parted/parted.c (_init): Do not emit this warning in script mode:
"You are not superuser.  Watch out for permissions."

Avoid "make distcheck" failure, with newer system headers.
* parted/parted.c: Include <limits.h>, for use of CHAR_MAX.

Updated Doxyfile

On the edge branch, labeled the version as 1.9.0 so we can mark it more correctly as the development tree.

Whitespace cleanups.

Whitespace cleanups.

Whitespace cleanups.

Whitespace cleanups.

Revert "Remove unused label."

This reverts commit bfd9a6d8b6322d870650b0e92bd936035ab28e76.

Patch from Jim Meyering <jim@meyering.net> to linux_write() to support logical

sector sizes other than 512 (PED_SECTOR_SIZE_DEFAULT).

Fix primary partition cylinder alignment error for DOS disk labels.

Fix off-by-one bug in parted when displaying information about the disk label (the disk size, specifically).

Do not translate partition names in 'parted print' command. This causes problems for non-Latin-based character sets. Also, we don't want to translate the partition name here as we are just showing what is in the disklabel, translating it distorts what the value is.

Also added some malloc checks in table.c.

Remove unused label.

Reverted my patch to configure.ac to remove -Wl,--as-needed from LDFLAGS when checking for libreadline.

libreadline is a problematic library. Autoconf adds a block to the generated configure script that checks to see if we should be passing the --as-needed flag to ld(1). There are portability reasons this code exists (best I can figure), but it presents a problem when we scan for libreadline.

libreadline specifically has unresolved symbols on at least Fedora and RHEL.  Why?  Well, the developer can choose to provide libtermcap, libncurses, or libncursesw (wide-char support) at compile time which all satisfy the curses API dependency that libreadline has.  When the parted configure script runs, it fails on readline because the --as-needed flag causes the linker to not include libncurses even though we already have that on the link line.

My solution is to remove the --as-needed flag when we scan for libreadline and then later check for rl_completion_matches() in libreadline.

More changes to set the version number to 1.8.3.

Set version to 1.8.3 for upcoming release.

[tests] Separate and improved test for reading of disk labels.

Fix endianness bugs

Add integration tests, then adjust code so that they pass.
These changes affect how --script (-s) works, and when diagnostics
are displayed.
* Makefile.am (SUBDIRS): Add tests.
* configure.ac (AC_OUTPUT): Add tests/Makefile.
* tests/Makefile.am: New file.
* tests/part-01, tests/label-01: New files.
* tests/part-02, tests/label-02: New files.
* parted/parted.c (enum) [PRETEND_INPUT_TTY]: Define.
Support a new, undocumented option: ---pretend-input-tty.
It is intended solely for testing.  The leading '---' ensures
that it will never conflict with a "regular" long option.
(do_mkpartfs): Prompt only when not in script mode.
Call ped_exception_leave_all() to decrement global counter.
Without this, the use of parted in tests/part-01 would still fail,
but would not issue the diagnostic explaining why.
* parted/ui.h (pretend_input_tty): Declare new global.
* parted/ui.c (exception_handler): Let the new "pretend_input_tty"
option override "!isatty (0)".
(init_ui): Don't set opt_script_mode.
* tests/lang-default, tests/priv-check: New files.

Temporarily disable the clone-dvh-label test.
* libparted/tests/label.c (test_clone_label):

[PATCH] Avoid warning from -Wformat-security; tweak error diagnostic.

* parted/parted.c (_parse_options): Upon failure, output diagnostic
to stderr, not stdout.  Use "program_name" value, not literal "parted".

Signed-off-by: Jim Meyering <jim@meyering.net>

[tests] New test for probing of disk labels.

Revert "[libparted] dvh label fixes"

This reverts commit dcf8e966abf997b9df69b1664c470ed2aa5c166b.

[libparted] dvh label fixes

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

[tests] Simplify test_clone_label removing the label reading test

Since the test_create_label already tests the label reading we
shouldn't redo it. It's always better to have simple tests so is much
easier to find the bugs.

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

[tests] Trivial coding style fixes around the tests code

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

[tests] Add _test_exception_handler to fail every time an exception is raised

Sometimes parts of code raises exceptions and this shouldn't happen on
tests. To ensure we catch them a specific exception handler has been
implemented.

The handler usage is very easy. You just need to put the following
code at testsuite main method:

...
        /* Fail when an exception is raised */
        ped_exception_set_handler (_test_exception_handler);
...

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

[tests] New test for cloning of disk labels.

[tests] Fix spacing around brackets and asterisks.

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

Handle options independent of their order.

Abort on any invalid option, and handle -v and -h first.

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

[tests] Small refactoring on test_create_label

 - Destroy the temporary device once the test has been finished,
 - Use the _implemented_disk_label and _create_disk_label methods
   to avoid code duplication.

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

[tests] Implements _implemented_disk_label and _create_disk_label on common.[hc]

 - _implemented_disk_label: returns 0 when we haven't yet implemented
   this specific label and 1 otherwise;

 - _create_disk_label: creates a disk label of a specific type on a
   specific device;

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

[tests] Ensure that we're able to read the just created disk label

To ensure we're able to read the just created disk label we introduced a call
ped_disk_new using the same loopback. This could detect bad label creating or
probing detectiong mistakes.

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

Indentation fixes in libparted/tests/.

Show the update /etc/fstab message only when there has been a change to the disk. The message is now shorter and more direct.

change the variable named start to offset to be consistent with the doc

remove any trace of PED_PARTITION_PRIMARY which was deprecated a long time ago to use PED_PARTITION_NORMAL

Added 'rescue', 'unit', and 'version' to the man page, plus some small edits.

Avoid segfault due a double free on reiserfs support (patch sent by Jakub Bogusz <qboosh@pld-linux.org>)

Make the wording of warning somewhat clearer.

Tweak some help messages: change third person to imperative for consistency, add single quotes and remove letters for clarity

Conditionalize the System Z stuff correctly.  We get headers installed on non-System Z platforms, but the code only compiles on that platform.

Remove the COMPILE_FOR_S390 flag-setting stuff.  It wasn't correct anyway.  Take a kernel-style approach and check uname -m to see if we are on s390.  If so, we build for zSeries.  Otherwise we do not.

Sleep for one second after uploading a file to ftp-upload.gnu.org.  Seems transactions need to be spaced out a bit otherwise the server will reject files.

Fix for compile problems with gcc-4.1.2 as indicated here:
http://lists.gnu.org/archive/html/bug-parted/2007-03/msg00008.html

Check for tgetent() in libtinfo as well as ncurses, curses, termcap, and termlib.

Remove some unused variables, check the return values from fgets() and asprintf(), and remove the unused _dm_remove_map() function.

Include vtoc.h for the format1_label_t typedef.

Remove GNU make-specific things from po4a.mk.

Install the libparted.pc file, not the libparted.pc.in file.

More spec file template cleanups.

Updated the release script to work with the new source layout.

Fixed faulty information in README-hacking (Bison is not a maintainer tool for Parted), and Makefile.maint (every project has certain variations of it).

po/POTFILES.in: Add lib/getopt.c

Prevented DASD specific headers and sources from being compiled/installed on non-s390 systems; and conditionally included fdasd.h in linux.h for s390 systems.

Move static variable flagging little endian or not to journal.c.  We don't want static variables in general, but if they must exist, put them in a C file, not a header.

Parameterize the little_endian flag for the macros in journal.h.

Modernize the spec file template.

Ignore more files.

Patch from Matthew Garrett <mjg59@srcf.ucam.org> to add HFS+ resize support.

hfs.c: Detect write failure.
* libparted/fs/hfs/hfs.c (hfs_extract_file, hfs_extract_bitmap):
(hfs_extract_mdb, hfsplus_extract_file, hfsplus_extract_vh):
Fail when fclose fails for a written-to file handle.

aix.c: Avoid memory overrun.  Don't assume logical sector size <= 512B
* libparted/labels/aix.c (struct AixLabel): Remove definition.
(aix_label_magic_get, aix_label_magic_set): New functions.
(read_sector): New function.
(aix_probe): Rewrite not to use the above, and not a static buffer.
(aix_clobber): Likewise.
Also, rather than PED_ASSERT'ing that aix_probe returns 1,
simply return 0 if aix_probe returns fails.
(ped_disk_aix_init): Remove assertion, now that AixLabel is gone.

Use PED_SECTOR_SIZE_DEFAULT not 512
* libparted/arch/beos.c (beos_read, beos_write):
* libparted/arch/linux.c (_device_get_length, _device_seek)
(linux_read, linux_write, _blkpg_add_partition):
* libparted/fs/fat/bootsector.c (fat_boot_sector_read):
* libparted/labels/bsd.c (bsd_alloc):
* libparted/labels/dos.c (probe_filesystem_for_geom):
* libparted/labels/rdb.c (amiga_read):

* parted/ui.c (non_interactive_mode): Plug a tiny leak.
Exercise by running the following with a CD-ROM in the drive:
valgrind --leak-check=full parted/parted /dev/cdrom print
Before, we'd leak the 6-byte command: "print\0".

linux.c: Avoid memory overrun.  Handle 2048-byte logical sectors.
* libparted/arch/linux.c (linux_read): Allocate the right amount of
space for the (potentially 2048-byte-long) sectors we're about to read.

Don't use deprecated mktemp, to avoid linker warning
* libparted/tests/common.c (_create_disk): Rewrite to
use mkstemp instead.

* libparted/tests/label.c: Plug a trivial leak.

Merge branch 'master' of git+ssh://rishi-guest@git.debian.org/git/parted/parted

Replaced [f]printf with fput[c|s] globally.

libparted/unit.c: constify interfaces
* include/parted/unit.h (PED_UNIT_LAST):
* libparted/unit.c (ped_unit_get_size):
(ped_unit_format_custom_byte, ped_unit_format_byte):
(ped_unit_format_custom, ped_unit_format, ped_unit_parse):
(find_suffix, parse_chs, clip, geometry_from_centre_radius):
(ped_unit_parse_custom):

Declare some static arrays to be "const".
* libparted/labels/dos.c (MBR_BOOT_CODE):
* libparted/labels/efi_crc32.c (crc32_tab):
* libparted/labels/pc98.c (MBR_BOOT_CODE):
* libparted/labels/vtoc.c (EBCtoASC, ASCtoEBC):

* configure.ac: Fix a typo: s/have_scintilla/have_check/

tests/label: Don't dereference NULL if open fails.
* libparted/tests/label.c (START_TEST): Return right away if
ped_device_get returns NULL.

Add "const" to parameters in these public interfaces.
* libparted/disk.c (ped_disk_check):
(ped_disk_get_primary_partition_count):
(ped_disk_get_last_partition_num, ped_partition_print):
(ped_disk_print):
(ped_partition_print): Also declare to be "static".
This is ok, since the function is referenced only from this file.
* include/parted/disk.h: Update their prototypes.

Use <config.h> consistently, not "config.h".
* Makefile.cfg (local-checks-to-skip): Remove sc_require_config_h,
to enable the "make distcheck" test for this.

start_pos is a PedSector, not a PedGeometry.

Preserve starting sector for primary NTFS 3.1 partitions (Windows Vista)
when modifying the DOS disk label.  NTFS 3.1 partitions do not start on
the 2nd head of the 1st cylinder at the beginning of the drive.

Patched parted.8 man page to show partition names apply to GPT disk labels

as well as Mac and PC98 disk labels.

Replaced [f]printf with fput[c|s] in libparted/fs/ext2/ext2_block_relocator.c.

Re-arranging .gitignore in alphabetical order.

Defined a dummy '_dump_history' function in parted/ui.c when Readline is absent. Thanks to Jim Meyering for the suggestion.

Added libparted/tests/label to .gitignore.

Fixed parted/ui.c to incorporate signal handlers for systems lacking 'sigaction', 'SA_SIGINFO', and other constants in signal.h.

Undo part of the dos.c constification.
* libparted/disk.c: The sole parameter of ped_disk_type_register
and ped_disk_type_unregister must *not* be const.
* libparted/labels/dos.c (msdos_disk_type): Likewise for the
corresponding static struct in dos.c, since those functions modify
the struct.  This didn't show up at compile time due to casts.
* include/parted/disk.h: Update prototypes, too.