Home - Waterfall Grid T-Grid Console Builders Recent Builds Buildslaves Changesources - JSON API - About

Builder diffutils-solaris10-i386 Build #175

Results:

Failed shell_3

SourceStamp:

Projectdiffutils
Repositorygit://git.savannah.gnu.org/diffutils.git
Branchmaster
Revision8da1f14d8406b477f8a64b71eb47518dc1b8726a
Got Revision8da1f14d8406b477f8a64b71eb47518dc1b8726a
Changes19 changes

BuildSlave:

unstable10x

Reason:

The SingleBranchScheduler scheduler named 'schedule-diffutils-solaris10-i386' triggered this build

Steps and Logfiles:

  1. git update ( 13 secs )
    1. stdio
  2. shell 'bash bootstrap' ( 7 mins, 56 secs )
    1. stdio
  3. shell_1 './configure --disable-gcc-warnings' ( 4 mins, 18 secs )
    1. stdio
    2. config.log
  4. shell_2 'gmake all' ( 4 mins, 49 secs )
    1. stdio
  5. shell_3 'gmake check' failed ( 2 mins, 5 secs )
    1. stdio
    2. gnulib-tests_test-suite.log
    3. tests_test-suite.log

Build Properties:

NameValueSource
branch master Build
builddir /export/home/buildbot/slave/diffutils-solaris10-i386 slave
buildername diffutils-solaris10-i386 Builder
buildnumber 175 Build
codebase Build
got_revision 8da1f14d8406b477f8a64b71eb47518dc1b8726a Git
project diffutils Build
repository git://git.savannah.gnu.org/diffutils.git Build
revision 8da1f14d8406b477f8a64b71eb47518dc1b8726a Build
scheduler schedule-diffutils-solaris10-i386 Scheduler
slavename unstable10x BuildSlave
workdir /export/home/buildbot/slave/diffutils-solaris10-i386 slave (deprecated)

Forced Build Properties:

NameLabelValue

Responsible Users:

  1. Bruno Haible
  2. Collin Funk
  3. Paul Eggert

Timing:

StartTue Jul 2 11:57:49 2024
EndTue Jul 2 12:17:13 2024
Elapsed19 mins, 23 secs

All Changes:

:

  1. Change #196897

    Category diffutils
    Changed by Paul Eggert <eggertohnoyoudont@cs.ucla.edu>
    Changed at Sun 12 May 2024 07:55:38
    Repository git://git.savannah.gnu.org/diffutils.git
    Project diffutils
    Branch master
    Revision 561caf873f4fc0ea9b9789a31feef6d67cebdc26

    Comments

    build: update gnulib submodule to latest

    Changed files

    • gnulib
  2. Change #196898

    Category diffutils
    Changed by Paul Eggert <eggertohnoyoudont@cs.ucla.edu>
    Changed at Sun 12 May 2024 07:55:38
    Repository git://git.savannah.gnu.org/diffutils.git
    Project diffutils
    Branch master
    Revision d6fc5a8ce142bd24af973ebf8b563512af3cac3c

    Comments

    maint: update bootstrap to Gnulib version

    Changed files

    • bootstrap
  3. Change #196899

    Category diffutils
    Changed by Paul Eggert <eggertohnoyoudont@cs.ucla.edu>
    Changed at Sun 12 May 2024 07:55:38
    Repository git://git.savannah.gnu.org/diffutils.git
    Project diffutils
    Branch master
    Revision a33c9b9e8578b3267f7c003ec6054d49e95ae178

    Comments

    diff: prefer stdbit to count-leading-zeros
    stdbit.h is standardized in C23, so use that instead of
    the GNU-specific count-leading-zeros module.
    * bootstrap.conf (gnulib_modules): Remove count-leading-zeros.
    Add stdbit.
    * src/system.h: Include stdbit.h instead of count-leading-zeros.h.
    (floor_log2): Implement via stdc_bit_width instead of via
    count_leading_zeros_ll.

    Changed files

    • bootstrap.conf
    • src/system.h
  4. Change #196938

    Category diffutils
    Changed by Paul Eggert <eggertohnoyoudont@cs.ucla.edu>
    Changed at Sun 12 May 2024 17:14:54
    Repository git://git.savannah.gnu.org/diffutils.git
    Project diffutils
    Branch master
    Revision cceca01aba9399769b80348d60fbef00f429bd99

    Comments

    maint: pacify GCC 14 -Wmissing-variable-declarations
    * src/Makefile.am (version.c): Have it include version.h,
    so that it checks its own interface.
    * src/diff.c (DIFF_INLINE): Define this instead of GDIFF_MAIN,
    for consistency with SYSTEM_INLINE.
    * src/diff.h (DIFF_EXTERN): New macro, replacing XTERN.
    All uses changed.  Adjust to GDIFF_MAIN→DIFF_INLINE change.
    * src/system.h (SYSTEM_EXTERN): Now takes an argument.
    All uses changed.

    Changed files

    • src/Makefile.am
    • src/diff.c
    • src/diff.h
    • src/system.h
  5. Change #197296

    Category diffutils
    Changed by Paul Eggert <eggertohnoyoudont@cs.ucla.edu>
    Changed at Wed 15 May 2024 00:50:12
    Repository git://git.savannah.gnu.org/diffutils.git
    Project diffutils
    Branch master
    Revision 195016c60f0b65886d4504287a4e99ad5c63f563

    Comments

    build: update gnulib submodule to latest

    Changed files

    • gnulib
  6. Change #197297

    Category diffutils
    Changed by Paul Eggert <eggertohnoyoudont@cs.ucla.edu>
    Changed at Wed 15 May 2024 00:50:12
    Repository git://git.savannah.gnu.org/diffutils.git
    Project diffutils
    Branch master
    Revision 62d075ad726e9a2a93c829873a63623284971c87

    Comments

    maint: be less fancy when defining extern vars
    Bug reported by Bruno Haible in <https://bugs.gnu.org/70951>.
    I never did like the XTERN macro and its descendants, as this
    “extension” to C causes more confusion than it cures, so let’s
    just get rid of it and use plain ‘extern’.
    * cfg.mk (_gl_TS_extern): Remove XTERN, SYSTEM_EXTERN.
    * src/Makefile.am (cmp_SOURCES, diff3_SOURCES, sdiff_SOURCES)
    (diff_SOURCES): Add system.c.
    * src/cmp.c, src/diff.c, src/diff3.c, src/sdiff.c (SYSTEM_INLINE):
    Remove.
    * src/diff.c: Define vars declared in diff.h.
    * src/diff.h (DIFF_EXTERN): Remove.  All uses removed.
    Just use ‘extern’ when declaring extern vars.
    * src/system.h (SYSTEM_EXTERN): Likewise.
    * src/system.c: New file.

    Changed files

    • cfg.mk
    • src/Makefile.am
    • src/cmp.c
    • src/diff.c
    • src/diff.h
    • src/diff3.c
    • src/sdiff.c
    • src/system.c
    • src/system.h
  7. Change #197405

    Category diffutils
    Changed by Paul Eggert <eggertohnoyoudont@cs.ucla.edu>
    Changed at Wed 15 May 2024 23:13:53
    Repository git://git.savannah.gnu.org/diffutils.git
    Project diffutils
    Branch master
    Revision 12dc6b3e6f39818c1ad971ca6971e9caa9b4a866

    Comments

    build: update gnulib submodule to latest

    Changed files

    • gnulib
  8. Change #197406

    Category diffutils
    Changed by Paul Eggert <eggertohnoyoudont@cs.ucla.edu>
    Changed at Wed 15 May 2024 23:13:53
    Repository git://git.savannah.gnu.org/diffutils.git
    Project diffutils
    Branch master
    Revision bf85e93a52366d5361cfad68e79667514259d666

    Comments

    maint: move same_file and stat_size into system.c
    Now that we have a system.c, there’s little point to these
    being extern inline functions.
    * src/system.c (same_file, stat_size): Move here from system.h,
    and make them ordinary extern functions.
    (proc_dev, symlink_size_ok): Now static, and private to their
    functions.
    * src/system.h (same_file, stat_size): Now just decls.

    Changed files

    • src/system.c
    • src/system.h
  9. Change #197462

    Category diffutils
    Changed by Paul Eggert <eggertohnoyoudont@cs.ucla.edu>
    Changed at Fri 17 May 2024 04:59:49
    Repository git://git.savannah.gnu.org/diffutils.git
    Project diffutils
    Branch master
    Revision 8f1af932956533018a01f0c8fbb659bd39680d2d

    Comments

    build: update gnulib submodule to latest

    Changed files

    • gnulib
  10. Change #197463

    Category diffutils
    Changed by Paul Eggert <eggertohnoyoudont@cs.ucla.edu>
    Changed at Fri 17 May 2024 04:59:49
    Repository git://git.savannah.gnu.org/diffutils.git
    Project diffutils
    Branch master
    Revision ba8e55270f8b4bb73fc07b1c5e043fb3975f3f2e

    Comments

    maint: adjust to Gnulib stdbit splitup
    * bootstrap.conf (gnulib_modules): Remove stdbit; add stdc_bit_width.

    Changed files

    • bootstrap.conf
  11. Change #197933

    Category diffutils
    Changed by Paul Eggert <eggertohnoyoudont@cs.ucla.edu>
    Changed at Tue 21 May 2024 19:08:53
    Repository git://git.savannah.gnu.org/diffutils.git
    Project diffutils
    Branch master
    Revision b8b3ac5c20da04786f16fd911cecaf39dcc0cae3

    Comments

    build: update gnulib submodule to latest

    Changed files

    • gnulib
  12. Change #199506

    Category diffutils
    Changed by Paul Eggert <eggertohnoyoudont@cs.ucla.edu>
    Changed at Tue 11 Jun 2024 18:12:35
    Repository git://git.savannah.gnu.org/diffutils.git
    Project diffutils
    Branch master
    Revision a9cca434611ee2293915110c74dc2c53ed4e58fd

    Comments

    maint: port to birthtime platforms
    Problem reported by Bruno Haible <https://bugs.gnu.org/71456>.
    * src/system.c (same_file): Fix typo on platforms with birthtime.

    Changed files

    • src/system.c
  13. Change #199509

    Category diffutils
    Changed by Paul Eggert <eggertohnoyoudont@cs.ucla.edu>
    Changed at Tue 11 Jun 2024 19:11:27
    Repository git://git.savannah.gnu.org/diffutils.git
    Project diffutils
    Branch master
    Revision d82aac15e6fc1e813d7f793e9cc3f82f9e593a44

    Comments

    maint: pacify ‘make sc_tight_scope’
    Problem reported by Bruno Haible <https://bugs.gnu.org/71455>.
    * src/system.h (same_file, stat_size): Add ‘extern’.

    Changed files

    • src/system.h
  14. Change #199520

    Category diffutils
    Changed by Paul Eggert <eggertohnoyoudont@cs.ucla.edu>
    Changed at Tue 11 Jun 2024 20:59:34
    Repository git://git.savannah.gnu.org/diffutils.git
    Project diffutils
    Branch master
    Revision 8f8ee605023d61298567be00671120bb85f6ab1c

    Comments

    maint: port ‘make distcheck’ to Ubuntu 24.04
    * configure.ac: Don’t use -Wsystem-headers, as <stdbit.h>
    uses static functions that couldn’t be called from
    extern inline functions if they were in user code.

    Changed files

    • configure.ac
  15. Change #199528

    Category diffutils
    Changed by Bruno Haible <brunoohnoyoudont@clisp.org>
    Changed at Wed 12 Jun 2024 00:21:13
    Repository git://git.savannah.gnu.org/diffutils.git
    Project diffutils
    Branch master
    Revision d7588ed7740ad345b4726009d81f817c703e93c6

    Comments

    tests: Fix ignore-case failure on several platforms
    * tests/ignore-case: Don't compare 'Ⓐ' with 'ⓐ'; this doesn't work on
    NetBSD 10.0 and Solaris 11.4. Don't compare 'Ꞻ' with 'ꞻ'; this doesn't
    work on CentOS 7, macOS, and Solaris 11 OmniOS.

    Changed files

    • tests/ignore-case
  16. Change #199540

    Category diffutils
    Changed by Paul Eggert <eggertohnoyoudont@cs.ucla.edu>
    Changed at Wed 12 Jun 2024 02:49:03
    Repository git://git.savannah.gnu.org/diffutils.git
    Project diffutils
    Branch master
    Revision d636f9c117d6383fedcc769661fa0fa4d372b153

    Comments

    diff: port to FreeBSD, NetBSD
    Problem reported by Bruno Haible <https://bugs.gnu.org/71486>.
    * src/diff.c (NOFOLLOW_SYMLINK_ERRNO): New constant.
    (compare_files): Use it instead of ELOOP.  When it is not ELOOP,
    treat ELOOP as a failure in resolving the parent directory;
    this saves a syscall in some situation.

    Changed files

    • src/diff.c
  17. Change #200018

    Category diffutils
    Changed by Paul Eggert <eggertohnoyoudont@cs.ucla.edu>
    Changed at Thu 20 Jun 2024 05:42:53
    Repository git://git.savannah.gnu.org/diffutils.git
    Project diffutils
    Branch master
    Revision 54c8930ca50953b995951ed185593a2831f5d335

    Comments

    maint: update .gitignore to add lib/stdbit.h

    Changed files

    • .gitignore
  18. Change #200019

    Category diffutils
    Changed by Paul Eggert <eggertohnoyoudont@cs.ucla.edu>
    Changed at Thu 20 Jun 2024 05:42:53
    Repository git://git.savannah.gnu.org/diffutils.git
    Project diffutils
    Branch master
    Revision da0c15f381f59e73dfc5e98403cad86fd707ee54

    Comments

    diff: avoid memory leak with --color-palette
    Problem found indirectly by Coverity, reported by Wasser Mai
    (Bug#71535).  Although the Coverity report was wrong, there was a
    different potential memory leak nearby.  Fix the problem by
    modifying the string in place, avoiding the need to call malloc.
    * src/util.c (color_ext_list, struct color_ext_type):
    Remove.  Not needed, as the list wasn’t used.
    All uses removed.
    (get_funky_string): Omit last argument output_count, as it’s
    easily calculated by caller.  This lets us call this function
    when we don’t care about the count.
    (color_palette): Now char *, not char const *, since we
    now update through it.
    (set_color_palette): Likewise.
    (parse_diff_color): Process color palette into itself, to avoid
    unnecessary malloc and free calls.  This pacifies Coverity, saves
    a bit of space in the normal case, and avoids a memory leak in
    some cases.  Do not process the palette twice, as its memory
    has been modified and this function had no effect on the
    color indicators the second time.

    Changed files

    • src/diff.h
    • src/util.c
  19. Change #200221

    Category diffutils
    Changed by Collin Funk <collin.funk1ohnoyoudont@gmail.com>
    Changed at Mon 24 Jun 2024 02:47:26
    Repository git://git.savannah.gnu.org/diffutils.git
    Project diffutils
    Branch master
    Revision 8da1f14d8406b477f8a64b71eb47518dc1b8726a

    Comments

    tests: don't fail when --program-transform-name is used
    Problem reported by Jun T <https://bugs.gnu.org/48062>.
    * tests/diff3: Use --diff-program=diff since --program-transform-name
    is applied after install.
    * tests/help-version (sdiff_setup, diff3_setup): Likewise.

    Changed files

    • tests/diff3
    • tests/help-version