Skip to content

Change shell interpreter to bash#26

Open
rokclimb15 wants to merge 1 commit intoaws:mainfrom
rokclimb15:patch-1
Open

Change shell interpreter to bash#26
rokclimb15 wants to merge 1 commit intoaws:mainfrom
rokclimb15:patch-1

Conversation

@rokclimb15
Copy link

Description

This file uses some purely bash syntax like function and will not execute correctly on a system with a different interpreter aliased to /bin/sh

Motivation and Context

Systems like Ubuntu don't have /bin/bash aliased to /bin/sh

Testing

./build64.sh now executes without error

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

License

  • By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

ORELASH added a commit to ORELASH/amazon-redshift-odbc-driver that referenced this pull request Feb 4, 2026
Updated version in both:
- version.txt: 2.1.14.0
- rsversion.h: 2,1,14

This ensures Windows can properly differentiate between driver versions
when upgrading, and users can verify they have the correct build installed.

Build aws#26 will have unique version: 2.1.14.0
MSI will be: AmazonRedshiftODBC64-Fork-v2.1.14.0-AzureOAuth.msi

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@vahid110
Copy link
Contributor

Hi @rokclimb15
Thank you for taking the time to improve the codebase.
I will discuss the changes with our team and get back to you shortly

@baa-ableton
Copy link

@rokclimb15 were you able to successfully build on Ubuntu with your change in 49433b4? I've been trying to install on Ubuntu and had more problems than just the shell type in the build script shebang.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants