simplesamlphp / simplesamlphp/simplesamlphp-module-discopower
Symlinks don't install correctly when Composer falls back on the PHP zip library
まだ誰も着手していません。
- 主要言語
- PHP
- スター
- 5
- フォーク
- 11
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
I couldn't figure out why the Discopower module wasn't installing correctly inside my Docker image of SSP. The symlinks under modules/discopower/public/assets/components were being installed as text files containing the path where they would have pointed had they been symlinks. This obviously didn't work so great when my browser tried to load the Jquery Javascript for Discopower.
After further investigation, I tried installing unzip into my Docker image before running composer, and that fixed it. I now have symlinks instead of text files. Clearly, the PHP zip library doesn't understand symlinks.
If possible, installation of this module should require that the unzip or 7z binary be installed on the system and not just the PHP zip library.
If that's not possible, the documentation should at least include this requirement for proper installation.
Ideally, these symlinks wouldn't be necessary, and the disco.twig code that loads these components would point to the files directly instead of the symlinks inside of the module.
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
まず、説明されている Docker セットアップで Composer のインストールを再現し、PHP の zip ライブラリと unzip または 7z を比較します。modules/discopower/public/assets/components と、これらのアセットを読み込む disco.twig のコードを調査します。インストールの動作または文書化されたシステム要件が symlinks を明確に扱っていれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- php
- 領域
- build-system
- issue の種類
- バグ
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- 説明が足りない
- 初心者へのやさしさ
- 35/100