Skip to content

README.md build instructions implicitly depend on testlogger, from MPWTest? #8

Description

@mstone

Checking out Objective-Smalltalk, MPWFoundation, selecting the 'stsh' scheme, and hitting build results in the following error:

...
    /bin/sh -c /Users/mstone/Library/Developer/Xcode/DerivedData/objective-smalltalk-ennpizgpendiubaaqiihhtetjsas/Build/Intermediates.noindex/MPWFoundation.build/Release/MPWFoundationFramework.build/Script-1FF8FE2809B49D0200652476.sh
---
/Users/mstone/Library/Developer/Xcode/DerivedData/objective-smalltalk-ennpizgpendiubaaqiihhtetjsas/Build/Intermediates.noindex/MPWFoundation.build/Release/MPWFoundationFramework.build/Script-1FF8FE2809B49D0200652476.sh: line 7: /Users/mstone/Library/Developer/Xcode/DerivedData/objective-smalltalk-ennpizgpendiubaaqiihhtetjsas/Build/Products/Release/MPWFoundation.framework: is a directory
Command PhaseScriptExecution failed with a nonzero exit code

Further printf-debugging in the script of the Test build phase of the MPWFoundationFramework target suggests that this may be due to tester=`which testlogger` failing to find testlogger in my PATH.

As for what to do: should MPWTest / testlogger be listed in the README as a required dependency, if the default build cannot complete without it?

(or, even better, can the default build configuration cause it to be built if needed without further developer intervention?)

Thanks!

Activity

  1. mpw commented on Jun 8, 2021

    @mpw
    Owner

    Thanks for the heads-up!

    This is supposed to be a soft dependency, meaning if you have MPWTest/testlogger installed on your system it runs the tests. If you don't, it just issues a warning that tests aren't being run. At least that's how it's supposed to work, and used to work, but now that logic appears to be broken.

    I am in the process of fixing it (and may just fold a slimmed-down MPWTest into MPWFoundation), but the easiest workaround for now is to either remove that build phase or disable it by checking the "For install builds only" checkbox.

  2. sebastianconcept commented on Aug 13, 2022

    @sebastianconcept

    @mpw how is that going?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions