Bug: Cleanup /tmp/pear

[glensc]

Describe the bug

It appears there's some cruft with fixed path in /tmp, owned by root, that can cause normal CI job to fail:

+ pecl install xhprof-0.9.4
WARNING: configuration download directory "/tmp/pear/install" is not writeable.  Change download_dir config variable to a writeable dir to avoid this warning
downloading xhprof-0.9.4.tgz ...
Starting to download xhprof-0.9.4.tgz (842,077 bytes)
........................................................................................................................................................................done: 842,077 bytes
ERROR: failed to write /usr/local/php/5.4/docs/xhprof/examples/.tmpsample.php: copy(/usr/local/php/5.4/docs/xhprof/examples/.tmpsample.php): failed to open stream: Permission denied

• https://github.com/perftools/php-profiler/actions/runs/12875396149/job/35896603274?pr=89

Here's full dump of /tmp:

• https://github.com/perftools/php-profiler/pull/89/checks

Version

• v2
• v1

Runners

GitHub Hosted

Operating systems

ubuntu-20.04

PHP versions

5.4, basically all 5.*

To Reproduce

      - name: Setup PHP
        uses: shivammathur/setup-php@v2
        with:
          php-version: "5.4"
          extensions: xdebug, xhprof, mongodb, tideways_xhprof

      - run: pecl install xhprof-0.9.4

Expected behavior

The /tmp is doesn't contain cruft from setup-php.

Screenshots/Logs

No response

Additional context

No response

Are you willing to submit a PR?

No

[shivammathur]

@glensc

setup-php does use /tmp directory to download builds and store temporary files for configuration during the run and for subsequent runs, cleaning them at the end will not solve your issue.

That error is because /tmp is owned by root on the runner by default. If you run pecl using sudo that should fix it, or you can change ownership of /tmp prior to running pecl.
Test workflow: https://github.com/shivammathur/test-setup-php/actions/runs/12876114013/workflow

Alternatively, you can specify xhprof-0.9.4 in extensions input and setup-php should install it correctly.
Test workflow: https://github.com/shivammathur/test-setup-php/actions/runs/12876131377/workflow

[glensc]

problem is not that /tmp is owned by root, but that /tmp/pear is owned by root and not cleaned up by setup-php

perhaps pear temporary directory should be reconfigured to use current user in path rather just /tmp/pear, like /tmp/pear-root

[glensc]

@shivammathur I hit this bug again:

• https://github.com/perftools/xhgui/actions/runs/16086429935/job/45398137106?pr=561

When trying to workaround another bug:

• Bug: setup-php lies about mongodb extension being installed #971

@shivammathur, why not just rm -rf /tmp/pear, so that user workflows are not affected by permission issue?