Skip to content

Commit

Permalink
Rewrite dependency lister - now produces *much* easier to read output
Browse files Browse the repository at this point in the history
Knowing where a module came from is quite valuable, but the segmentation done
in cebc0cc makes it rather hard finding a particular module. Instead add
an extra prefix with shorthand "dirs of interest" names, and putput the entire
list in ci-alphabetical order.

Additionally make sure we explicitly print out our versions under CI (a silly
omission from earlier), and explicitly omit loading dist_* optdeps

The rough list of changes:
  - Explicitly report expected but failed-to-load modules
  - Add knowledge of './t' => T, under same rules as LIB
  - Add support for (site|vendor)lib_stem (whatever that is)
  - Add support for (site|vendor)prefix
  - Add support for ./blib/(lib|arch)
  - Add support for an explicit CWD marker
  - When no sourcing is found - report the @inc index if possible
  - If a module came from {SVP}{AL}, list it even if no version is defined
  - Explain missing checksums
  - Display the contents of @inc, making it even easier to follow sourcing

While the diff is massive, the actual logic did not change in any significant
way.
  • Loading branch information
ribasushi committed Aug 12, 2015
1 parent 62b92f8 commit 8336115
Show file tree
Hide file tree
Showing 4 changed files with 512 additions and 215 deletions.
2 changes: 1 addition & 1 deletion lib/DBIx/Class.pm
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ our $VERSION;
# $VERSION declaration must stay up here, ahead of any other package
# declarations, as to not confuse various modules attempting to determine
# this ones version, whether that be s.c.o. or Module::Metadata, etc
$VERSION = '0.082899_01';
$VERSION = '0.082899_15';

$VERSION = eval $VERSION if $VERSION =~ /_/; # numify for warning-free dev releases

Expand Down
4 changes: 2 additions & 2 deletions maint/travis-ci_scripts/30_before_script.bash
Original file line number Diff line number Diff line change
Expand Up @@ -144,8 +144,8 @@ else
fi

echo_err "$(tstamp) Dependency installation finished"
# this will display list of available versions
perl Makefile.PL

run_or_err "Re-configure" "perl Makefile.PL"

# make sure we got everything we need
if [[ -n "$(make listdeps)" ]] ; then
Expand Down
3 changes: 3 additions & 0 deletions maint/travis-ci_scripts/40_script.bash
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,9 @@ run_harness_tests() {
make test 2> >(tee "$TEST_STDERR_LOG")
}

# announce everything we have on this box
TRAVIS="" perl -Ilib t/00describe_environment.t >/dev/null

TEST_T0=$SECONDS
if [[ "$CLEANTEST" = "true" ]] ; then
echo_err "$(tstamp) Running tests with plain \`make test\`"
Expand Down
Loading

0 comments on commit 8336115

Please sign in to comment.