Skip to content
New issue

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

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

Already on GitHub? # to your account

[SPE-4405] Normalize source input location path between Windows and Unix #1048

Open
nielthiart opened this issue Oct 30, 2024 · 1 comment
Open
Labels
CLI Created by Linear-GitHub Sync linear Low priority Created by Linear-GitHub Sync platform Created by Linear-GitHub Sync
Milestone

Comments

@nielthiart
Copy link
Contributor

nielthiart commented Oct 30, 2024

When a new SDK is created in a Windows environment, the sources[*].inputs[*].location in .speakeasy/workflow.yaml uses backslash in relative paths to address the input file.

If the same SDK is later regenerated on macOS/Linux using speaskeasy run, the input location can't be resolved.

Steps

  1. Use Windows
  2. Create a new SDK using speakeasy quickstart, referencing a local OpenAPI document as source.
  3. Transfer parent folder of SDK with OpenAPI document and SDK to Linux/macOS.
  4. On Linux/macOS, in the SDK directory, run speakeasy run

Expected result

SDK regenerates from the OpenAPI document in the parent folder.

Actual result

speakeasy run
failed to validate target test-django: failed to validate source TestDjango-OAS: failed to get output location: input file ..\openapi.yaml does not exist

Workaround

Updating the path in .speakeasy/workflow.yaml to use slashes instead of backslashes resolves this for macOS/Linux users.

SPE-4405

@simplesagar
Copy link
Member

Thanks for the report Niel !

@simplesagar simplesagar changed the title Normalize source input location path between Windows and Unix [SPE-4405] Normalize source input location path between Windows and Unix Oct 30, 2024
@simplesagar simplesagar added CLI Created by Linear-GitHub Sync platform Created by Linear-GitHub Sync Medium priority Created by Linear-GitHub Sync labels Oct 30, 2024
@simplesagar simplesagar modified the milestones: v.124, v.125 Nov 11, 2024
@simplesagar simplesagar modified the milestones: v.125, v.126 Nov 20, 2024
@simplesagar simplesagar modified the milestones: v.126, v.127 Nov 27, 2024
@simplesagar simplesagar modified the milestones: v.127, v.128, v.129 Dec 4, 2024
@simplesagar simplesagar modified the milestones: v.129, v.130 Dec 18, 2024
@simplesagar simplesagar added Low priority Created by Linear-GitHub Sync and removed Medium priority Created by Linear-GitHub Sync labels Dec 20, 2024
@simplesagar simplesagar modified the milestones: v.130, v.131 Dec 25, 2024
@simplesagar simplesagar modified the milestones: v.131, v.132, v.133 Jan 1, 2025
@simplesagar simplesagar modified the milestones: v.133, v.134 Jan 15, 2025
@simplesagar simplesagar removed this from the v.134 milestone Jan 22, 2025
@simplesagar simplesagar added this to the v.135 milestone Jan 22, 2025
@simplesagar simplesagar modified the milestones: v.135, v.136, v.137 Jan 29, 2025
@simplesagar simplesagar modified the milestones: v.137, v.138, v.139 Feb 12, 2025
@simplesagar simplesagar modified the milestones: v.139, v.140, v.141 Feb 26, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
CLI Created by Linear-GitHub Sync linear Low priority Created by Linear-GitHub Sync platform Created by Linear-GitHub Sync
Projects
None yet
Development

No branches or pull requests

2 participants