emacs-china / elpa

Emacs China ELPA 镜像
https://elpamirror.emacs-china.org/
154 stars 15 forks source link

Change the name of the configuration-layer--elpa-archives #42

Closed appleshan closed 6 years ago

xuchunyang commented 6 years ago

请无视失败了的 Travis Check,忘记给 PR 关了,刚才已经关掉了。

我不了解 Spacemacs:所有的 Spacemacs 用户现在都应该用这个新的变量吗?旧的不能工作了吗?

appleshan commented 6 years ago

现在,所有的 Spacemacs 用户都应该用这个新的变量了。

xuchunyang commented 6 years ago

但我没在 Spacemacs 的代码里搜到(至少是在 master)?

https://github.com/syl20bnr/spacemacs/search?q=configuration-layer-elpa-archives&type=Code&utf8=%E2%9C%93

旧的不能工作了吗?

还有这个问题(改了名字,没有向后兼容吗?)。

appleshan commented 6 years ago

严格来讲: 在 develop 中,此变量才刚刚改名,旧的不能工作了; 在 master 中,此变量没有改名,旧的可以工作。 真心纠结!

xuchunyang commented 6 years ago

那就是不向后兼容的意思了(用户选项改名要做向后兼容的,但考虑到原来是个私有变量,不做也正常)。

所以至于这个 PR,要合并的话,最好等到上游 master 分支包含这个修改。

colin4124 commented 6 years ago

我觉得可以在 Spacemacs 下单独注明下 develop 的用法。 我自己被这个问题困扰了很久,希望同样使用 develop 分支的用户可以避免这个问题。

xuchunyang commented 6 years ago

同意,但这个 PR 目前还没有这么做。

另外,一开始用户就有定制需求的话,就应该及时告诉开发者不要用私有变量名,一年之后再改就会有现在的麻烦。

xuchunyang commented 6 years ago

已经合并了 @colin4124 的 3bfe901,效果见:

https://elpa.emacs-china.org/ 上的得等一会儿再更新,因为不是自动的,得手动更。

xuchunyang commented 6 years ago

Done.