Skip to content

feat: Enable PIE (Position Indepenet Executable) - #442

Merged
pbukva merged 2 commits into
masterfrom
fix/enable_PIE
Sep 10, 2026
Merged

pbukva merged 2 commits into
masterfrom
fix/enable_PIE

Conversation

@pbukva

@pbukva pbukva commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

Proposed Changes

Enabling PIE in go compiler options.
The PIE feature is always enabled if the fetchd binary is built using all Makefile targets related to fetchd build (build, build-linux and install).

Linked Issues

[if applicable, add links to issues resolved by this PR]

Types of changes

What type of change does this pull request make (put an x in the boxes that apply)?

  • Bug fix (non-breaking change that fixes an issue).
  • New feature added (non-breaking change that adds functionality).
  • Breaking change (fix or feature that would cause existing functionality to stop working as expected).
  • Documentation update.
  • Something else (e.g., tests, scripts, example, deployment, infrastructure).

Checklist

Put an x in the boxes that apply:

  • I have read the CONTRIBUTING guide
  • Checks and tests pass locally

If applicable

  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that code coverage does not decrease
  • I have added/updated the documentation

Further comments

This change does not carry any changes in the source code. It only enables go compiler PIE feature.

@pbukva pbukva self-assigned this Sep 10, 2026
@github-actions

github-actions Bot commented Sep 10, 2026 •

Copy link
Copy Markdown

Visit the preview URL for this PR (updated for commit 0b7b500):

https://fetch-docs-preview--pr442-fix-enable-pie-e34ysmj0.web.app

(expires Tue, 15 Sep 2026 11:43:49 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: f2de39fd4e81249941960b74fbab0a62d90d69f8

MissingNO57
MissingNO57 previously approved these changes Sep 10, 2026

@MissingNO57 MissingNO57 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@MissingNO57 MissingNO57 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pbukva
pbukva merged commit c0acd83 into master Sep 10, 2026
4 checks passed
@pbukva
pbukva deleted the fix/enable_PIE branch September 10, 2026 11:58
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