[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.

Hide two unnecessarily-"extern" functions.
* libparted/labels/mac.c (mac_alloc): Declare static.
* libparted/labels/dos.c (raw_part_parse): Declare static.

* libparted/labels/dos.c: Complete constification of this file.
* libparted/disk.c (ped_disk_type_register): Make the sole
parameter "const", so that caller's argument may be "const".
(ped_disk_type_unregister): Likewise.

* po/POTFILES.in: Add partprobe/partprobe.c, now that it contains
translatable strings.

Make partprobe accept --help and --version options.
Add long options: --no-update (same as existing -d), and
--summary (same as existing -s).
* partprobe/partprobe.c
Include configmake.h, getopt.h, and NLS-related things.
(main): Rewrite option handling.
Along the way, fix a bug whereby "partprobe DEV1 DEV2 ... DEVN" would
exit successfully whenever process_dev (DEVN) returns nonzero, even
when that function fails for each of the preceding devices.

Make clearfat accept --help and --version options.
* debug/clearfat/clearfat.c (AUTHORS, PROGRAM_NAME): Define.
Include gettext-related things.
Include closeout.h.
(usage): New function.
(main): Set up for translations, use close_stdout.
* bootstrap.conf: Add long-options, for clearfat.
* debug/clearfat/Makefile.am: Use gnulib.

* include/parted/disk.h (struct _PedDiskOps) [write]: Change
parameter type to be "const".

Begin making libparted/labels/*.c const-correct.
For an introduction, see
http://mail-archive.com/parted-devel@lists.alioth.debian.org/msg00930.html
In particular, I've used a const-adding case in mac.c to work around
the current, inconsistent semantics of mac_write.

Avoid a libtool warning:
libtool: link: warning: `-release' is ignored for convenience libraries
* libparted/labels/Makefile.am (liblabels_la_LDFLAGS): Don't define.

Make "make distcheck" work better.
* Makefile.cfg (local-checks-to-skip): Skip sc_system_h_headers, too.
* Makefile.maint (my-distcheck): Don't depend on $(release_archive_dir).

Avoid recursive "LIBS = ... $(LIBS)" assignments.
* libparted/labels/Makefile.am (liblabels_la_LIBADD): Add $(INTLLIBS)
here rather than via a recursive LIBS = ... $(LIBS) assignment.
* libparted/fs/Makefile.am (libfs_la_LIBADD): Likewise.

Use $(VAR), rather than obsolescent @VAR@ automake notation.

Enable the makefile-check rule.
* Makefile.cfg: Remove makefile-check from the list of rules to skip.
* Makefile.am: change e.g., @VERSION@ to $(VERSION).
* debug/clearfat/Makefile.am: Likewise.
* libparted/Makefile.am: Likewise.
* libparted/fs/Makefile.am: Likewise.
* libparted/fs/amiga/Makefile.am: Likewise.
* libparted/fs/bfs/Makefile.am: Likewise.
* libparted/fs/ext2/Makefile.am: Likewise.
* libparted/fs/fat/Makefile.am: Likewise.
* libparted/fs/hfs/Makefile.am: Likewise.
* libparted/fs/jfs/Makefile.am: Likewise.
* libparted/fs/linux_swap/Makefile.am: Likewise.
* libparted/fs/ntfs/Makefile.am: Likewise.
* libparted/fs/reiserfs/Makefile.am: Likewise.
* libparted/fs/ufs/Makefile.am: Likewise.
* libparted/fs/xfs/Makefile.am: Likewise.
* libparted/labels/Makefile.am: Likewise.
* libparted/tests/Makefile.am: Likewise.
* parted/Makefile.am: Likewise.
* partprobe/Makefile.am: Likewise.

The above conversion caused a problem:
* libparted/Makefile.am: Remove this recursive definition:
LIBS = @INTLLIBS@ @LIBS@.  Once converted to LIBS = ...$(LIBS),
GNU make would (rightly) fail, objecting to the recursive definition.
(libparted_la_LIBADD): Instead, add $(INTLLIBS) here.

Enable the po-check rule.

* Makefile.cfg (local-checks-to-skip): Remove po-check.
* po/POTFILES.in: Add a bunch of names of files that contain
translatable strings.  Remove some which contain none.

Add infrastructure for more rigorous "make distcheck" rules.

* GNUmakefile: New file.
* .prev-version: New file.
* Makefile.am (MAINTAINERCLEANFILES): Remove names of files now in build-aux/.
(distcheck-hook): New rule, to run the checks in Makefile.maint.
(EXTRA_DIST): Add .prev-version.
* Makefile.maint: New file.  Improved "make distcheck" rules.
* Makefile.cfg: Select which Makefile.maint checks to run.
(local-checks-to-skip): List all currently-failing and irrelevant
tests here.
* build-aux/vc-list-files: New file.  List version-controlled files.
* .gitignore: Remove build-aux, since now it contains a
version-controlled file.

Fixing indentation in parted/ui.c. Replaced TABs with spaces to ensure uniformity in all editors.

Fixing indentation in parted/ui.c.

Normalize the way we handle a generated-and-distributed file, parted.spec.
* Makefile.am (EXTRA_DIST): Add parted.spec.
(parted.spec): New rule.
(MAINTAINERCLEANFILES): Add parted.spec.
* configure.ac (AC_OUTPUT): Remove parted.spec from the list.

Add -I$(top_srcdir)/lib, required for new use of close_stdout.
* partprobe/Makefile.am (partedincludedir):

Arrange for "make distcheck" to be stricter.

* bootstrap: new file.

Fixing libparted/arch/gnu.c to remove build-time errors.

partprobe: Detect/report any error when writing to stdout.
* partprobe/Makefile.am (partprobe_LDADD): Add lib/libparted.la.
* partprobe/partprobe.c:
Include "closeout.h" for close_stdout.
Include "version-etc.h" for version_etc prototype.
Declare global, program_name.
(PROGRAM_NAME, AUTHORS): Define.
(help): Use PROGRAM_NAME, rather than hard-coding it.
(version): Use gnulib's version_etc.
(main): Set program_name.
Use atexit to close stdout carefully upon exit.

Add a bunch of .gitignore files.

Avoid a gcc format warning:
* debug/clearfat/clearfat.c (_do_help): Use fputs, not printf.

Remove now-generated files.
* po/ca.po, po/cs.po, po/da.po, po/de.po, po/es.po, po/fr.po:
* po/gl.po, po/id.po, po/it.po, po/ja.po, po/nl.po, po/nn.po:
* po/pl.po, po/pt.po, po/pt_BR.po, po/ru.po:
* po/sv.po, po/tr.po, po/uk.po, po/vi.po, po/zh_CN.po, po/zh_TW.po:
* po/Makevars: Remove now-generated file.
* po/parted.pot: Remove -generated file.

mv parted.m4 into new dir, m4/

* Makefile.am (EXTRA_DIST): Remove parted.m4.
Now that it's in m4, it's included automatically.
(aclocal_DATA): Remove unnecessary definition.

* libparted/tests/label.c: Fail if _create_disk returns NULL.
Make global and functions static.
* libparted/tests/common.c (_create_disk): Handle fopen failure.
Handle fclose failure.

Include <config.h> from every .c file:
* libparted/unit.c, libparted/timer.c:
* libparted/cs/natmath.c, libparted/cs/constraint.c:
* libparted/tests/label.c, libparted/tests/common.c:
* libparted/fs/fat/count.c, libparted/fs/fat/bootsector.c:
* libparted/fs/fat/context.c, libparted/fs/fat/fatio.c:
* libparted/fs/fat/fat.c, libparted/fs/fat/clstdup.c:
* libparted/fs/fat/resize.c, libparted/fs/fat/traverse.c:
* libparted/fs/fat/table.c, libparted/fs/fat/calc.c:
* libparted/fs/amiga/amiga.c, libparted/labels/fdasd.c:
* libparted/labels/dvh.c, libparted/labels/efi_crc32.c:
* libparted/labels/vtoc.c, parted/command.c:
* libparted/tests/label.c, libparted/tests/common.c:

* libparted/libparted.c: Include "configmake.h" for LOCALEDIR defn.

parted: Detect/report stdout write errors.
* parted/parted.c: Include "closeout.h" for close_stdout.
Include "configmake.h" for LOCALEDIR definition.
Include "version-etc.h" for version_etc prototype.
Declare global, program_name.
(PROGRAM_NAME, AUTHORS): Define.
(_version): Use gnulib's version_etc.
(main): Set program_name.
Use atexit to close stdout carefully upon exit.

Include <config.h>, not "../config.h".

Begin adjusting Makefile.am files to use gnulib.
* libparted/Makefile.am (partedincludedir): Add -I$(top_srcdir)/lib.
* parted/Makefile.am (parted_LDADD): Add lib/libparted.la.
(partedincludedir): Add -I$(top_srcdir)/lib.

Update to use gnulib, and newer autoconf, automake conventions.

Add build-from-checkout instructions.
* README-hacking: New file.  Mostly copied from coreutils'
README-cvs.
* bootstrap: Generate ChangeLog here, if needed/possible,
before invoking tools that require its presence.
* autogen.sh: Remove this file.  It's superseded by bootstrap.

Use gnulib, modernize some uses of autoconf, automake, etc.
* bootstrap, bootstrap.conf: New files.

* configure.ac: Modernize.
Require the latest stable releases: autoconf-2.61, automake-1.10.
Require gettext-0.15.
Use more modern form of AC_INIT, etc.
Now that the version string must be hard-coded in the AC_INIT line,
add code to ensure it agrees with the PED_* and LT_* variables.
Use AM_CPPFLAGS, not CFLAGS
Remove obsolete setting of ALL_LINGUAS.
Don't set CFLAGS=-D_GNU_SOURCE=1 explicitly , since that's already
done via AC_GNU_SOURCE, which gnulib pulls in by default,
through gl_EARLY.
Use gl_INIT and gl_EARLY.

* Makefile.am (SUBDIRS): Add lib.
(EXTRA_DIST): Remove names that are automatically included.
* lib/Makefile.am: New file.

Fix memleaks. Patch thanks to Mike Hommey