Skip to content

Conversation

@weidongxu-microsoft
Copy link
Contributor

@weidongxu-microsoft weidongxu-microsoft commented Mar 19, 2025

Temporary use http-specs dev version.

There is 1 bug in TCGC, but it likely won't affect service. Azure/typespec-azure#2411

E2e test added

autorest.java Azure/autorest.java#3047

@microsoft-github-policy-service microsoft-github-policy-service bot added the emitter:client:java Issue for the Java client emitter: @typespec/http-client-java label Mar 19, 2025
@github-actions
Copy link
Contributor

No changes needing a change description found.

@azure-sdk
Copy link
Collaborator

azure-sdk commented Mar 19, 2025

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

@weidongxu-microsoft weidongxu-microsoft changed the title WIP http-client-java, upgrade compiler 0.67 http-client-java, upgrade compiler 0.67 Mar 20, 2025
@weidongxu-microsoft weidongxu-microsoft force-pushed the http-client-java_upgrade-compiler-0.67 branch from 9f1c19c to 7fe5761 Compare March 20, 2025 06:16
@weidongxu-microsoft weidongxu-microsoft force-pushed the http-client-java_upgrade-compiler-0.67 branch 3 times, most recently from d0ac723 to ed8adcb Compare March 20, 2025 13:02
@weidongxu-microsoft
Copy link
Contributor Author

weidongxu-microsoft commented Mar 20, 2025

The output of tsp compile seems triggering error or exist in Powershell, which I still have no idea why...
https://dev.azure.com/azure-sdk/public/_build/results?buildId=4673089&view=logs&j=1e8d8958-901b-5118-6ce2-6cb3891ef470&t=63d0b491-266a-5256-b92d-a20247d2ec64&l=461

Write-Error: /mnt/vss/_work/_temp/4afa686f-b7f2-4b98-bf95-619d3dc720e3.ps1:3
Line |
   3 |  . '/mnt/vss/_work/1/s/packages/http-client-java/eng/scripts/Test-Pack …
     |  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Code generation failed: - Compiling...

I had to pipe the output to null to workaround
bee0598


But it also seems related to Powershell (or, maybe the handling of output in DevOps). I am not able to reproduce such thing in my local...

@weidongxu-microsoft weidongxu-microsoft force-pushed the http-client-java_upgrade-compiler-0.67 branch from 9ed269f to bee0598 Compare March 20, 2025 15:29
@weidongxu-microsoft weidongxu-microsoft marked this pull request as ready for review March 20, 2025 16:03
Copy link
Member

@XiaofeiCao XiaofeiCao left a comment

Choose a reason for hiding this comment

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

May need a test code regen.

@weidongxu-microsoft weidongxu-microsoft added this pull request to the merge queue Mar 21, 2025
Merged via the queue into microsoft:main with commit 00181e8 Mar 21, 2025
25 checks passed
@weidongxu-microsoft weidongxu-microsoft deleted the http-client-java_upgrade-compiler-0.67 branch March 21, 2025 04:34
@weidongxu-microsoft
Copy link
Contributor Author

The output of tsp compile seems triggering error or exist in Powershell, which I still have no idea why... https://dev.azure.com/azure-sdk/public/_build/results?buildId=4673089&view=logs&j=1e8d8958-901b-5118-6ce2-6cb3891ef470&t=63d0b491-266a-5256-b92d-a20247d2ec64&l=461

Write-Error: /mnt/vss/_work/_temp/4afa686f-b7f2-4b98-bf95-619d3dc720e3.ps1:3
Line |
   3 |  . '/mnt/vss/_work/1/s/packages/http-client-java/eng/scripts/Test-Pack …
     |  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Code generation failed: - Compiling...

I had to pipe the output to null to workaround bee0598

But it also seems related to Powershell (or, maybe the handling of output in DevOps). I am not able to reproduce such thing in my local...

This likely due to #6662

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

Labels

emitter:client:java Issue for the Java client emitter: @typespec/http-client-java

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants