Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecate bundles, optional install stanzas, and dependent stanzas #113

Closed
pjf opened this issue Oct 21, 2014 · 1 comment · Fixed by #209
Closed

Deprecate bundles, optional install stanzas, and dependent stanzas #113

pjf opened this issue Oct 21, 2014 · 1 comment · Fixed by #209
Assignees
Labels
Core (ckan.dll) Issues affecting the core part of CKAN Enhancement New features or functionality Policy Issues with our policy Spec Issues affecting the spec

Comments

@pjf
Copy link
Member

pjf commented Oct 21, 2014

After writing this post and combined with #110 and #112, there doesn't seem to be a great need for bundled mods, or optional sections.

This even works for previously thorny things like RSS bundling CustomBiomes with its own maps. We simply have CustomBiomes depend upon CustomBiomesData, and that can be provided by either CustomBiomesKerbal (which uses the same file as CustomBiomes), or RealSolarSystem.

This may mean that we'll want to support embedded CKAN/NetCKAN files which describe multiple mods that come from the same file, so things don't end up going out of sync in the future.

@pjf pjf added Core (ckan.dll) Issues affecting the core part of CKAN design Enhancement New features or functionality ★★☆ Policy Issues with our policy Spec Issues affecting the spec and removed design labels Oct 21, 2014
@pjf pjf added In progress We're still working on this and removed ★★☆ labels Oct 31, 2014
@pjf pjf self-assigned this Oct 31, 2014
@pjf
Copy link
Member Author

pjf commented Oct 31, 2014

Working on this now. :)

pjf added a commit to pjf/CKAN that referenced this issue Oct 31, 2014
Updates the spec, the schema, and the code. Includes tests.

Closes KSP-CKAN#113 (deprecate things).
Closes KSP-CKAN#129 (add filter keywords).

This will require an update to CKAN-meta, which I've already got ready.
@pjf pjf added pull request and removed In progress We're still working on this labels Oct 31, 2014
@pjf pjf reopened this Oct 31, 2014
pjf added a commit to pjf/CKAN that referenced this issue Oct 31, 2014
Updates the spec, the schema, and the code. Includes tests.

Closes KSP-CKAN#113 (deprecate things).
Closes KSP-CKAN#129 (add filter keywords).

This will require an update to CKAN-meta, which I've already got ready.
@pjf pjf closed this as completed in 5a5e9ee Oct 31, 2014
RichardLake pushed a commit to RichardLake/CKAN that referenced this issue May 30, 2015
Updates the spec, the schema, and the code. Includes tests.

Closes KSP-CKAN#113 (deprecate things).
Closes KSP-CKAN#129 (add filter keywords).

This will require an update to CKAN-meta, which I've already got ready.
RichardLake added a commit to RichardLake/CKAN that referenced this issue May 30, 2015
Lower "not removing directory" messages to info level.
RichardLake pushed a commit to RichardLake/CKAN that referenced this issue May 30, 2015
Show incompatible but installed mods in the GUI
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Core (ckan.dll) Issues affecting the core part of CKAN Enhancement New features or functionality Policy Issues with our policy Spec Issues affecting the spec
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants