[PR #1] Rewrite and cleanup with more modern methods, safer shell constructs, etc. #1

Open
opened 2026-02-28 00:20:24 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/tcude/vmware-to-proxmox-migration-script/pull/1
Author: @desrod
Created: 3/7/2024
Status: 🔄 Open

Base: mainHead: shell-cleanup-and-simplify


📝 Commits (1)

  • a8b6f9a First pass at a cleaner rewrite with modern bash constructs, quoting shell vars, protecting read() against backslashes, removing 'function' where redundant, other cleanups

📊 Changes

1 file changed (+94 additions, -93 deletions)

View changed files

📝 linux_migration.sh (+94 -93)

📄 Description

First pass at a cleaner rewrite with modern bash constructs, quoting shell vars, protecting read() against backslashes, removing 'function' where redundant, other cleanups.

Please test it to make sure I didn't break anything.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/tcude/vmware-to-proxmox-migration-script/pull/1 **Author:** [@desrod](https://github.com/desrod) **Created:** 3/7/2024 **Status:** 🔄 Open **Base:** `main` ← **Head:** `shell-cleanup-and-simplify` --- ### 📝 Commits (1) - [`a8b6f9a`](https://github.com/tcude/vmware-to-proxmox-migration-script/commit/a8b6f9a1a9b6251dbb725d898862c27b7d409a0e) First pass at a cleaner rewrite with modern bash constructs, quoting shell vars, protecting read() against backslashes, removing 'function' where redundant, other cleanups ### 📊 Changes **1 file changed** (+94 additions, -93 deletions) <details> <summary>View changed files</summary> 📝 `linux_migration.sh` (+94 -93) </details> ### 📄 Description First pass at a cleaner rewrite with modern bash constructs, quoting shell vars, protecting read() against backslashes, removing 'function' where redundant, other cleanups. _Please_ test it to make sure I didn't break anything. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
Sign in to join this conversation.
No labels
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/vmware-to-proxmox-migration-script#1
No description provided.