Page MenuHomePhorge

Skip writing symbol cache to non-writable directories
Needs RevisionPublic

Authored by xtex on Tue, Feb 11, 03:22.
Tags
None
Referenced Files
F2992421: D25870.1740216727.diff
Fri, Feb 21, 09:32
F2991597: D25870.1740194602.diff
Fri, Feb 21, 03:23
F2990187: D25870.1740171966.diff
Thu, Feb 20, 21:06
F2985523: D25870.1740063517.diff
Wed, Feb 19, 14:58
F2985371: D25870.1740039247.diff
Wed, Feb 19, 08:14
F2984338: D25870.1740010804.diff
Wed, Feb 19, 00:20
F2981662: D25870.1739921795.diff
Mon, Feb 17, 23:36
F2979384: D25870.1739849311.diff
Mon, Feb 17, 03:28
Tokens
"Yellow Medal" token, awarded by valerio.bozzolan.

Details

Reviewers
valerio.bozzolan
Group Reviewers
O1: Blessed Committers
Summary

As the title, this makes it possible to install arcanist to
non-writable paths, for example, when installing arcanist with system
package managers.

Test Plan

Put arcanist to /usr/lib/arcanist (:

Diff Detail

Repository
rARC Arcanist
Branch
fix-cache
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 1692
Build 1692: arc lint + arc unit

Event Timeline

xtex requested review of this revision.Tue, Feb 11, 03:22

(1. Are you an heroic package manager? <3 nice to know!)

  1. Are you using arc liberate as test plan? Maybe we can mention it. So I think you get this exception:
[2025-02-11 06:39:39] EXCEPTION: (FilesystemException) Requested path '/usr/lib/arcanist/src/__phutil_library_map__.php' is not writable. at [<arcanist>/src/filesystem/Filesystem.php:1288]
  1. In case, probably it's better to package Arcanist only after the command arc liberate was executed successfully with sufficient write permissions. Then, later calls should probably not auto-trigger it - if I remember correctly.

What do you think about? Thaanks :)

( I was running arc lint in phorge.git and got the same error.
Thank u for your recommendation of running arc liberate. Will add that to packaging script :)

valerio.bozzolan requested changes to this revision.EditedTue, Feb 11, 06:53

OK super! <3

Feel free to share your packages in the chat so we can put you in the right contact pages for eternal glory lol

(I flag this change as "more background + more test plan needed" - but feel free to update anytime or share further comments)

This revision now requires changes to proceed.Tue, Feb 11, 06:53

https://github.com/AOSC-Dev/aosc-os-abbs/pull/9638

Running arc liberate before installing worked well for me. Thank you again!