Skip to content

Conversation

@claudiamurialdo
Copy link
Collaborator

Set the right FileVersion attribute on assembly files (it was always 0).

…ribute on assembly files which was always 0.
@claudiamurialdo claudiamurialdo requested a review from a team as a code owner March 20, 2023 19:24
@genexusbot
Copy link
Collaborator

Cherry pick to beta success

@claudiamurialdo claudiamurialdo temporarily deployed to external-storage-tests March 20, 2023 19:24 — with GitHub Actions Inactive
…_ENV var are not seen by the step that defined the var.

It should be set a power shell var instead of a local var ( $Env:COMMIT_NUMBER = @($(git....)  or keep msbuild explicit parameter /p:COMMIT_NUMBER=$COMMIT_NUMBER
@genexusbot
Copy link
Collaborator

Cherry pick to beta success

@claudiamurialdo claudiamurialdo temporarily deployed to external-storage-tests March 20, 2023 20:10 — with GitHub Actions Inactive
Copy link
Member

@jechague jechague left a comment

Choose a reason for hiding this comment

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

LGTM

@claudiamurialdo claudiamurialdo merged commit f176e71 into master Mar 20, 2023
@claudiamurialdo claudiamurialdo deleted the commit-number-to-file-version branch March 20, 2023 21:15
claudiamurialdo added a commit that referenced this pull request Mar 20, 2023
* Define COMMIT_NUMBER environment var to set the right FileVersion attribute on assembly files which was always 0.

* Keep COMMIT_NUMBER local var as a msbuild parameter given that GITHUB_ENV var are not seen by the step that defined the var.
It should be set a power shell var instead of a local var ( $Env:COMMIT_NUMBER = @($(git....)  or keep msbuild explicit parameter /p:COMMIT_NUMBER=$COMMIT_NUMBER

(cherry picked from commit f176e71)
claudiamurialdo added a commit that referenced this pull request Mar 20, 2023
* Define COMMIT_NUMBER environment var to set the right FileVersion attribute on assembly files which was always 0.

* Keep COMMIT_NUMBER local var as a msbuild parameter given that GITHUB_ENV var are not seen by the step that defined the var.
It should be set a power shell var instead of a local var ( $Env:COMMIT_NUMBER = @($(git....)  or keep msbuild explicit parameter /p:COMMIT_NUMBER=$COMMIT_NUMBER

(cherry picked from commit f176e71)
claudiamurialdo added a commit that referenced this pull request Mar 20, 2023
* Define COMMIT_NUMBER environment var to set the right FileVersion attribute on assembly files which was always 0.

* Keep COMMIT_NUMBER local var as a msbuild parameter given that GITHUB_ENV var are not seen by the step that defined the var.
It should be set a power shell var instead of a local var ( $Env:COMMIT_NUMBER = @($(git....)  or keep msbuild explicit parameter /p:COMMIT_NUMBER=$COMMIT_NUMBER

(cherry picked from commit f176e71)
claudiamurialdo added a commit that referenced this pull request Mar 20, 2023
* Define COMMIT_NUMBER environment var to set the right FileVersion attribute on assembly files which was always 0.

* Keep COMMIT_NUMBER local var as a msbuild parameter given that GITHUB_ENV var are not seen by the step that defined the var.
It should be set a power shell var instead of a local var ( $Env:COMMIT_NUMBER = @($(git....)  or keep msbuild explicit parameter /p:COMMIT_NUMBER=$COMMIT_NUMBER

(cherry picked from commit f176e71)
claudiamurialdo added a commit that referenced this pull request Mar 20, 2023
* Define COMMIT_NUMBER environment var to set the right FileVersion attribute on assembly files which was always 0.

* Keep COMMIT_NUMBER local var as a msbuild parameter given that GITHUB_ENV var are not seen by the step that defined the var.
It should be set a power shell var instead of a local var ( $Env:COMMIT_NUMBER = @($(git....)  or keep msbuild explicit parameter /p:COMMIT_NUMBER=$COMMIT_NUMBER

(cherry picked from commit f176e71)
claudiamurialdo added a commit that referenced this pull request Jul 25, 2025
claudiamurialdo added a commit that referenced this pull request Jul 25, 2025
(cherry picked from commit 238ce5f)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants