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

Check if bash is available #11930

Closed
wants to merge 8 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 2 additions & 3 deletions .github/workflows/gui-changed-files.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,9 +36,8 @@ jobs:
.prettierrc.js
.prettierignore
vitest.workspace.ts
.github/workflows/gui-changed-files.yml
.github/workflows/gui-checks.yml
.github/workflows/gui-pull-request.yml
.github/workflows/gui*
.github/workflows/ide*
.github/workflows/storybook.yml
files_ignore: |
app/ide-desktop/**
Expand Down
40 changes: 40 additions & 0 deletions .github/workflows/ide-packaging.yml
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,46 @@ jobs:
- self-hosted
- Windows
steps:
- name: Check Git and bash locations
shell: pwsh
run: |
Write-Host "Checking Git installation..." -ForegroundColor Yellow
$gitExe = (Get-Command git.exe).Path
Write-Host "Git executable: $gitExe"

Write-Host "`nGit version:" -ForegroundColor Yellow
git --version

Write-Host "`nChecking for bash in Git directory..." -ForegroundColor Yellow
$gitDir = Split-Path $gitExe -Parent
$expectedBashPath = Join-Path (Split-Path $gitDir -Parent) "bin\bash.exe"

if (Test-Path $expectedBashPath) {
Write-Host "Found bash.exe at: $expectedBashPath"
& $expectedBashPath --version
} else {
Write-Host "bash.exe not found at expected path: $expectedBashPath"
}

Write-Host "`nAll bash.exe locations in PATH:" -ForegroundColor Yellow
Get-Command bash.exe -ErrorAction SilentlyContinue | Select-Object -ExpandProperty Path
- name: Add Git bin to PATH and check bash
shell: cmd
run: |
echo "Original PATH:"
echo %PATH%
echo.
echo "Adding Git bin to PATH..."
set "PATH=C:\Program Files\Git\bin;%PATH%"
echo.
echo "New PATH:"
echo %PATH%
echo.
echo "Checking for bash.exe..."
where bash.exe
echo.
echo "Verifying bash version:"
bash --version
- name: Setup bazel environment
uses: bazel-contrib/setup-bazel@09f3a72d13a081857b0ee94e986ffa84caef7c85
with:
Expand Down
1 change: 1 addition & 0 deletions app/gui/src/project-view/App.vue
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,7 @@ watch(
{ immediate: true },
)

// This comment runs CI
useEventListener(window, 'beforeunload', () => logger.send('ide_project_closed'))

const appConfig = computed(() => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -198,8 +198,10 @@ type Redshift_Dialect
Dialect_Flag.Supports_Infinity -> True
Dialect_Flag.Case_Sensitive_Text_Comparison -> True
Dialect_Flag.Supports_Sort_Digits_As_Numbers -> False
Dialect_Flag.Case_Insensitive_Ordering -> False
Dialect_Flag.Supports_Case_Insensitive_Ordering -> False
Dialect_Flag.Order_By_Unicode_Normalization_By_Default -> True
Dialect_Flag.Order_By_Unicode_Normalization_When_Case_Insensitive -> True
Dialect_Flag.Case_Insensitive_Ordering_By_Default -> True
Dialect_Flag.Allows_Mixed_Type_Comparisons -> False
Dialect_Flag.Supports_Unicode_Normalization -> False
Dialect_Flag.NaN_Non_Comparable -> True
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -34,10 +34,16 @@ type Dialect_Flag
Supports_Sort_Digits_As_Numbers
## PRIAVTE
Specifies if the backend supports case insensitive ordering.
Case_Insensitive_Ordering
Supports_Case_Insensitive_Ordering
## PRIVATE
Specifies if the backend supports unicode normalization in its default ordering.
Specifies if the backend uses case insensitive sort in its default ordering.
Case_Insensitive_Ordering_By_Default
## PRIVATE
Specifies if the backend uses unicode normalization in its default ordering.
Order_By_Unicode_Normalization_By_Default
## PRIVATE
Specifies if the backend uses unicode normalization when doing case insentive ordering.
Order_By_Unicode_Normalization_When_Case_Insensitive
## PRIVATE
Specifies if mixed operations comparing
mixed types are allowed by a given backend. Some backends will allow
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -270,8 +270,10 @@ type Postgres_Dialect
Dialect_Flag.Supports_Infinity -> True
Dialect_Flag.Case_Sensitive_Text_Comparison -> True
Dialect_Flag.Supports_Sort_Digits_As_Numbers -> False
Dialect_Flag.Case_Insensitive_Ordering -> True
Dialect_Flag.Supports_Case_Insensitive_Ordering -> True
Dialect_Flag.Order_By_Unicode_Normalization_By_Default -> True
Dialect_Flag.Order_By_Unicode_Normalization_When_Case_Insensitive -> True
Dialect_Flag.Case_Insensitive_Ordering_By_Default -> True
Dialect_Flag.Allows_Mixed_Type_Comparisons -> False
Dialect_Flag.Supports_Unicode_Normalization -> False
Dialect_Flag.NaN_Non_Comparable -> False
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -281,8 +281,10 @@ type SQLite_Dialect
Dialect_Flag.Supports_Infinity -> True
Dialect_Flag.Case_Sensitive_Text_Comparison -> True
Dialect_Flag.Supports_Sort_Digits_As_Numbers -> False
Dialect_Flag.Case_Insensitive_Ordering -> False
Dialect_Flag.Supports_Case_Insensitive_Ordering -> True
Dialect_Flag.Order_By_Unicode_Normalization_By_Default -> False
Dialect_Flag.Order_By_Unicode_Normalization_When_Case_Insensitive -> False
Dialect_Flag.Case_Insensitive_Ordering_By_Default -> False
Dialect_Flag.Allows_Mixed_Type_Comparisons -> True
Dialect_Flag.Supports_Unicode_Normalization -> False
Dialect_Flag.NaN_Non_Comparable -> True
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -259,8 +259,10 @@ type SQLServer_Dialect
Dialect_Flag.Supports_Infinity -> False
Dialect_Flag.Case_Sensitive_Text_Comparison -> False
Dialect_Flag.Supports_Sort_Digits_As_Numbers -> False
Dialect_Flag.Case_Insensitive_Ordering -> True
Dialect_Flag.Supports_Case_Insensitive_Ordering -> True
Dialect_Flag.Order_By_Unicode_Normalization_By_Default -> True
Dialect_Flag.Order_By_Unicode_Normalization_When_Case_Insensitive -> True
Dialect_Flag.Case_Insensitive_Ordering_By_Default -> True
Dialect_Flag.Allows_Mixed_Type_Comparisons -> False
Dialect_Flag.Supports_Unicode_Normalization -> False
Dialect_Flag.NaN_Non_Comparable -> True
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -259,8 +259,10 @@ type Snowflake_Dialect
Dialect_Flag.Supports_Infinity -> True
Dialect_Flag.Case_Sensitive_Text_Comparison -> True
Dialect_Flag.Supports_Sort_Digits_As_Numbers -> False
Dialect_Flag.Case_Insensitive_Ordering -> True
Dialect_Flag.Order_By_Unicode_Normalization_By_Default -> True
Dialect_Flag.Supports_Case_Insensitive_Ordering -> True
Dialect_Flag.Order_By_Unicode_Normalization_By_Default -> False
Dialect_Flag.Order_By_Unicode_Normalization_When_Case_Insensitive -> True
Dialect_Flag.Case_Insensitive_Ordering_By_Default -> False
Dialect_Flag.Allows_Mixed_Type_Comparisons -> False
Dialect_Flag.Supports_Unicode_Normalization -> False
Dialect_Flag.NaN_Non_Comparable -> False
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -267,9 +267,9 @@ add_order_specs suite_builder setup =
t2.at "delta" . to_vector . should_equal ["a03", "a1", "a10", "a2"]
t2.at "alpha" . to_vector . should_equal [0, 2, 3, 1]

if setup.flagged ..Case_Insensitive_Ordering then group_builder.specify "should support case insensitive ordering" <|
if setup.flagged ..Supports_Case_Insensitive_Ordering then group_builder.specify "should support case insensitive ordering" <|
t1 = data.table.sort [..Name "eta"] text_ordering=(..Case_Insensitive)
expected = case setup.flagged ..Order_By_Unicode_Normalization_By_Default of
expected = case setup.flagged ..Order_By_Unicode_Normalization_When_Case_Insensitive of
False -> ["Aleph", "alpha", "Beta", "bądź"]
True -> ["Aleph", "alpha", "bądź", "Beta"]
t1.at "eta" . to_vector . should_equal expected
Expand Down Expand Up @@ -298,7 +298,7 @@ add_order_specs suite_builder setup =
t3.at "psi" . to_vector . should_equal [Nothing, "c01", "c10", "C2"]

t4 = data.table.sort [..Name "psi"]
case setup.flagged ..Case_Insensitive_Ordering of
case setup.flagged ..Case_Insensitive_Ordering_By_Default of
True -> t4.at "psi" . to_vector . should_equal [Nothing, "c01", "c10", "C2"]
False -> t4.at "psi" . to_vector . should_equal [Nothing, "C2", "c01", "c10"]

Expand Down
2 changes: 1 addition & 1 deletion test/Table_Tests/src/Database/Upload_Spec.enso
Original file line number Diff line number Diff line change
Expand Up @@ -242,7 +242,7 @@ add_specs suite_builder setup make_new_connection persistent_connector=True =
group_builder.specify "should not issue a DELETE statement for the original table name in dry run mode, even if the table does not exist" <|
original_table_name = Name_Generator.random_name "no-delete-test"

log_file = enso_project.data / "transient" / "sql.log"
log_file = (Project_Description.new enso_dev.Table_Tests).data / "transient" / "sql.log"
log_file.delete_if_exists
Test_Environment.unsafe_with_environment_override "ENSO_SQL_LOG_PATH" log_file.absolute.path <|
Context.Output.with_disabled <|
Expand Down
2 changes: 1 addition & 1 deletion test/Table_Tests/src/In_Memory/Common_Spec.enso
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ in_memory_setup =
True
flagged_fn flag:Dialect_Flag =
case flag of
Dialect_Flag.Case_Insensitive_Ordering -> False
Dialect_Flag.Case_Insensitive_Ordering_By_Default -> False
Dialect_Flag.Removes_Trailing_Whitespace_Casting_From_Char_To_Varchar -> False
_ -> True

Expand Down
Loading