[PR #165] [MERGED] Use of 'grep -o' doesn't work on SunOS (fixes #164) #741

New Issue

adam · 2025-12-29T01:28:44+01:00

adam commented

2025-12-29 01:28:44 +01:00

📋 Pull Request Information

Original PR: https://github.com/dehydrated-io/dehydrated/pull/165
Author: @bahamat
Created: 3/9/2016
Status: ✅ Merged
Merged: 3/22/2016
Merged by: @lukas2511

Base: master ← Head: portability

📝 Commits (1)

4b8883b Use of 'grep -o' doesn't work on SunOS (fixes #164)

📊 Changes

1 file changed (+5 additions, -4 deletions)

View changed files

📝 letsencrypt.sh (+5 -4)

📄 Description

Fixes #164.

Change some calls from grep -o to sed. These changes have been tested on SmartOS in the GZ, SmartOS in a zone with pkgsrc, Mac OS X and Linux (Ubuntu 14.04).

This allows letsencrypt.sh to be used from the global zone on SmartOS without requiring bootstrapping pkgsrc or installing additional tools.

_{🔄 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/dehydrated-io/dehydrated/pull/165 **Author:** [@bahamat](https://github.com/bahamat) **Created:** 3/9/2016 **Status:** ✅ Merged **Merged:** 3/22/2016 **Merged by:** [@lukas2511](https://github.com/lukas2511) **Base:** `master` ← **Head:** `portability` --- ### 📝 Commits (1) - [`4b8883b`](https://github.com/dehydrated-io/dehydrated/commit/4b8883b47afb80a04bcd2758641176d3d736578e) Use of 'grep -o' doesn't work on SunOS (fixes #164) ### 📊 Changes **1 file changed** (+5 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `letsencrypt.sh` (+5 -4) </details> ### 📄 Description Fixes #164. Change some calls from `grep -o` to `sed`. These changes have been tested on SmartOS in the GZ, SmartOS in a zone with pkgsrc, Mac OS X and Linux (Ubuntu 14.04). This allows `letsencrypt.sh` to be used from the global zone on SmartOS without requiring bootstrapping pkgsrc or installing additional tools. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

adam added the pull-request label 2025-12-29 01:28:44 +01:00

adam closed this issue

2025-12-29 01:28:45 +01:00

adam referenced this issue

2025-12-29 01:29:59 +01:00

[PR #741] [CLOSED] Fix typo in comment #915

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/dehydrated#741