ModificationStation / 1.7.3-LTS

A modification of MCP to provide a "Long Term Service" patch for Minecraft Beta 1.7.3
40 stars 15 forks source link

Preconfigured Workspace #18

Closed calmilamsy closed 5 years ago

calmilamsy commented 5 years ago

Removes some of the legwork when creating a workspace. All you have to do is open /src as an eclipse workspace now.

All client libraries and natives are all preconfigured. Both projects will appear automagically in your workspace.

Will need to be tested on different configurations first.

mineLdiver commented 5 years ago

Wait, why? There is already a workspace in the /eclipse/ folder

calmilamsy commented 5 years ago

Wait, why? There is already a workspace in the /eclipse/ folder

  1. They arent automatically copied.
  2. They have incorrect paths.
mineLdiver commented 5 years ago

Wait, why? There is already a workspace in the /eclipse/ folder

1. They arent automatically copied.

2. They have incorrect paths.
  1. Copied where?
  2. Wdym incorrect paths? Works perfectly for me.
calmilamsy commented 5 years ago

-snip-

  1. Copied where?
  2. Wdym incorrect paths? Works perfectly for me.
  1. Copied from /eclipse to /src
  2. They refer from / when /src/<project> is the cwd, unless I am missing something.
mineLdiver commented 5 years ago

-snip-

  1. Copied where?
  2. Wdym incorrect paths? Works perfectly for me.
1. Copied from `/eclipse` to `/src`

2. They refer from `/` when `/src/<project>` is the cwd, unless I am missing something.
  1. Why copying? You can just open /eclipse/ as a workspace and it'll work.
  2. From above, nothing is broken if you just open /eclipse/ as workspace and not /src/
calmilamsy commented 5 years ago

I stand corrected. I never knew that you could just use /eclipse as a workspace. Should probably edit readme to say open /eclipse as a workspace.

Closing.

Johnanater commented 5 years ago

Deleting branch...

Dereku commented 5 years ago

Preconfigured Workspace is a good idea. But a Eclipse's workspace - that's bad. If you really want to do that - look at the Maven/Gradle

Johnanater commented 5 years ago

Indeed

mineLdiver commented 5 years ago

Yep, gradle/maven for LTS would be a good addition.