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

Sync bash-4.4 with bash-5.2 #57

Merged
merged 8 commits into from
Dec 30, 2024
Merged

Sync bash-4.4 with bash-5.2 #57

merged 8 commits into from
Dec 30, 2024

Conversation

jansorg
Copy link
Collaborator

@jansorg jansorg commented Dec 27, 2024

This syncs with the latest changes of the 5.2 branch. I hope that I didn't break anything...
I briefly tested this and it worked fine.

Update: AFAIK tests are failing because of a change in the 4.4 Docker image. I've reported this at tianon/docker-bash#44

@jansorg jansorg requested a review from rocky December 27, 2024 16:29
rocky and others added 8 commits December 30, 2024 14:45
* Resolve file paths in "backtrace"
* Save mapping when doing the "load" command; resolve the filename in  the "backtrace" command
* Add a test for the "load" command

---------

Co-authored-by: Joachim Ansorg <[email protected]>
In particular, resolve the initial source directory and store that as
_Dbg_init_dir. Resolve the name of main bash program and add that to
canonicalized filenames.
Co-authored-by: Joachim Ansorg <[email protected]>
Co-authored-by: Joachim Ansorg <[email protected]>
Co-authored-by: Joachim Ansorg <[email protected]>
In some cases though automatic file loading no longer happens.
In particular, on "list" and "break" commands.
@jansorg jansorg force-pushed the jansorg/sync-4.4-5.2 branch from 0f763d9 to 71f6189 Compare December 30, 2024 13:45
@rocky rocky merged commit 248cb7d into bash-4.4 Dec 30, 2024
1 check passed
@jansorg jansorg deleted the jansorg/sync-4.4-5.2 branch December 30, 2024 13:54
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.

2 participants