project-x icon indicating copy to clipboard operation
project-x copied to clipboard

project-x on MEPLA

Open manufactory opened this issue 3 years ago • 12 comments

Hello,

Is there any specific reason why project-x is not available on MEPLA? If you have nothing against it, I'd be happy to create recipe and open a PR to get it added :)

manufactory avatar Nov 16 '21 10:11 manufactory

Hey @manufactory,

I was initially hesitant to submit this to MELPA since project.el is being actively developed and thus project-x might break often. However there have been no breaking changes since I wrote it, so it might be a good idea to do so now. I don't have the time for this at the moment, but feel free to submit it to MELPA if you're interested. The code will need to be linted to pass the MELPA review, I can make you a co-author if you're okay with doing that.

What do you think?

EDIT: You can also submit the linted version as a PR I guess.

karthink avatar Nov 17 '21 03:11 karthink

Hey,

Thanks! I ran all the linters suggested by MELPA and they found no issues.

When going through the code, the only thing that I noticed is that (project-root) can be replaced by (project-root) provided by project.el.

I found an issue when playing around: Projects (sometimes?) don't get recognized when there's a .project file in the main directory. My test case is quite simple: I created a directory, ~/tmp/project-x-test, containing two files and an empty .project file, invoked (project-x-mode) and the project directory is always my home directory, not ~/tmp/project-x-test. I'm also quite busy, but perhaps I find some time to look into that. It might just be the order in which packages or hooks are loaded. I'll keep you posted :)

manufactory avatar Nov 23 '21 10:11 manufactory

Any update on this? It would be lovely to see this package available publicly. I may be able to look into it in the next week or two if you're both busy. :)

kingcons avatar Jan 23 '22 19:01 kingcons

I'm currently to busy to work on this, so if @karthink is fine with it go ahead :)

manufactory avatar Jan 25 '22 11:01 manufactory

A little held up at the moment, please feel free to submit it. 👍

On Sun, Jan 23, 2022, 11:42 AM Brit Butler @.***> wrote:

Any update on this? It would be lovely to see this package available publicly. I may be able to look into it in the next week or two if you're both busy. :)

— Reply to this email directly, view it on GitHub https://github.com/karthink/project-x/issues/5#issuecomment-1019553439, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACBVOLC4QKNDPUDPKP7DXWTUXRKZNANCNFSM5ID5LBIA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you commented.Message ID: @.***>

karthink avatar Jan 25 '22 18:01 karthink

I volunteer to submit it, @manufactory

trev-dev avatar Apr 25 '23 02:04 trev-dev

@trev-dev Thanks. Before submission, it might be a good idea to update the package for Emacs 29, since the latest project.el has changed a bit. Nothing's broken in project-x (as best I can tell), but project.el has gained some of project-x's features, such as the ability to identify projects based on the existence of a special file in the root directory of the project.

I'm not on 29 yet so I haven't looked into it.

karthink avatar Apr 25 '23 23:04 karthink

Funny, as the main reason I want project-x is for that project root feature. Emacs as a graphical program is a tad annoying to set up with the proper context.

Do you suspect that Emacs 29 will essentially deprecate project-x?

trev-dev avatar Apr 27 '23 16:04 trev-dev

Do you suspect that Emacs 29 will essentially deprecate project-x?

project-x's other feature is to save and restore your project files and window configuration. project.el is not gaining this feature in Emacs 29, so project-x won't be deprecated. However there are many packages for saving and restoring state, such as tab-bookmark, burly, perspective, the built-in desktop.el etc. So I wouldn't say it's essential.

karthink avatar Apr 27 '23 16:04 karthink

Apologies @karthink, but due to reasons not in your control I'm moving on from Emacs for now.

trev-dev avatar May 04 '23 04:05 trev-dev

This is a great package, I think it should go on melpa. I'm pretty new to emacs so I can't do a pr for it though unfortunately. Btw I'm on 29.2 and it works well.

orazdow avatar Feb 04 '24 22:02 orazdow

This is a great package, I think it should go on melpa. I'm pretty new to emacs so I can't do a pr for it though unfortunately. Btw I'm on 29.2 and it works well.

If you're interested please go ahead, the only changes project-x will need are stylistic/linter warnings.

karthink avatar Feb 05 '24 22:02 karthink