Skip to content
This repository was archived by the owner on Jul 27, 2023. It is now read-only.

Swift: Preserve build path between build and run stages#733

Merged
Kamran64 merged 4 commits intomasterfrom
swift-file-path
Mar 27, 2020
Merged

Swift: Preserve build path between build and run stages#733
Kamran64 merged 4 commits intomasterfrom
swift-file-path

Conversation

@djones6
Copy link
Contributor

@djones6 djones6 commented Mar 26, 2020

Avoids swift #file paths being invalid when used to look up relative paths to resources.

This was resulting in Kitura's FileResourceServer logging errors (due to a design flaw) when the initial landing page is viewed after embedding an existing Kitura project into a project based on the swift stack.

Checklist:

Modifying an existing stack:

  • Updated the stack version in stack.yaml

Contributing a new stack:

  • Describe how application dependencies are managed:

  • Explain how Appsody file watcher is utilized:

  • Describe other Appsody environment variables defined by the stack image:

  • Describe any limitations and known issues:

Related Issues:

Avoids swift `#file` paths being invalid when used to look up relative paths to resources.
@djones6 djones6 requested a review from ianpartridge March 26, 2020 15:18
ianpartridge
ianpartridge previously approved these changes Mar 26, 2020
ianpartridge
ianpartridge previously approved these changes Mar 27, 2020
Copy link
Collaborator

@Kamran64 Kamran64 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@djones6 - Please can you bump the version in the stack.yaml - then this can be merged :)

Copy link
Collaborator

@Kamran64 Kamran64 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Thanks.

@Kamran64 Kamran64 merged commit 92bdfcb into master Mar 27, 2020
@djones6 djones6 deleted the swift-file-path branch March 27, 2020 11:01
@helenmasters
Copy link
Contributor

helenmasters commented Mar 27, 2020

I've used the newly released swift stack v0.2.6 and verified that this fixes the errors that I was seeing logged from Kitura's FileResourceServer.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants