Koutaro Mukai
Koutaro Mukai
It took a lot of geometric functions, but it seems to be working initially. The absolute position of the pad is not changed, and the new center point (the small...
ありがとうございます。おっしゃる通りレジストリを見に行くのは実用的ではなさそうです。 しかし村上さんの「最後のスラッシュがなくても、そのディレクトリがある場合は、ディレクトリの指定として扱われるべきでしょう。」という発言からはいっそう、`theme: 'css'`と指定されていれば`css`ディレクトリを見に行くべきだと感じられます……。 > - npm registryに`css`というパッケージが存在するかを確認 > - 存在しない、もしくはVivliostyle Themeパッケージでない場合、ローカル環境のパッケージを使用 この実装案が少しわからなかったのですが、素朴な案として「npm-package-argにかける**前に**、パスとして解釈して存在するか・Vivliostyle Themeパッケージか判別する」処理を挟むのは不都合がありそうでしょうか?
> theme はあくまでnpm packageを指定するためのオプションのため、npmの名前解決のルールに例外的な処理を加えることで将来的なバグの原因になることを懸念しています。 正直なところ、バグの原因になりそうなのは同意します。 `npm install ~`の指定と同じルールと考えると理屈はわかりますが、カレントディレクトリ直下の場合のみ接頭辞的に`./`が必要なのは、現状の`vivliostyle.config.js`にある`// .css or local dir or npm package.`からは読み取れないように思います。文字通りには、先にディレクトリ名として考慮してくれることを期待します。実装は変えずこの説明を正確にするのでもよいと思います。 実装を変える場合、カレントディレクトリの`foo`というディレクトリを意図して`theme: "foo"`と書いたが実際には`foo`ディレクトリが存在しないかつnpmレジストリにも存在しない場合には、「npm registryに`foo`というパッケージが存在しない」というエラーではなく、「ディレクトリも、レジストリにパッケージも存在しない」というエラーを出せば原因の特定はむしろ容易になりそうです。実現可能かはわかりません ```mermaid flowchart LR s0{"そのディレクトリにnpmパッケージが存在する?"} s1{"(既存の処理)"} s2["パッケージを使用する"] s3["エラー「ディレクトリも、レジストリにパッケージも存在しない」"] s0 -- yes --> s2...
実装を複雑にしたくないのは十分理解いたします。`theme`の解決方法自体は変えずに、`theme`に何を記述できるのかのドキュメントを補う形で進めていければよいのかなと思っています。現状想定されているのは - `*.css` URLおよびローカルファイルへのパス - ローカルのnpmパッケージへのパス(ただし`/`を含む) - `npm install ~`相当のパッケージ指定 という感じでしょうか。 > インストールしたパッケージのpackage.jsonに`vivliostyle`フィールドがあるかどうかをチェックするなどで判定できます。 パッケージがVivliostyleテーマかどうか判定できるようにするのはこの件に限らず役に立つかもしれませんが、既存のテーマへの影響が気にかかります。
村上さんにも賛同していただいた、ディレクトリ内にpackage.jsonが存在する場合に`theme: "foo"`をカレントディレクトリの`foo/`と解釈するという挙動は、`foo`はNode.jsパッケージであるけれどVivliostyleテーマとして使用したいわけではない、npmレジストリの`foo`を使用したい場合に回避策がなくなることに気づきました。 また起草した私自身が`./foo`と書く事情に納得できるようになっており、この件のパッチを作成するモチベーションがなくなっています。 加えて #626 で取り込んでいただいた[テーマをローカルディレクトリに作成する例](https://github.com/vivliostyle/vivliostyle-cli/tree/main/examples/local-theme)で、必要な指定については説明できたと感じています。このIssueは私からクローズとさせてください。
I agree with your point about the possibility of people misunderstanding.
@owlstronaut Sorry, but I'm not entirely sure if PR #8436 actually resolves this issue. It’s true that `npm install --install-links ../b` now appears to succeed. However, once the code in...
@owlstronaut Thank you for the follow-up. Unfortunately, the issue still seems to persist. As shown below, it looks like the same error occurs. ``` $ npm --version 11.5.2 $ mkdir...
@owlstronaut Thank you for the fix! This indeed solved my issue. By the way, is it by design that transitive external file dependencies are only handled when `--install-links` is used?...
After further investigation into the git history, I discovered that the MSMF backend supported camera enumeration before pygame 2.6.0. This functionality was likely removed in #4192. The name-based camera enumeration...