Skip to content

Remove impossible unix buildconstraint#95

Merged
Mzack9999 merged 1 commit intoprojectdiscovery:mainfrom
iamleot:permission_linux_only_linux
Feb 26, 2023
Merged

Remove impossible unix buildconstraint#95
Mzack9999 merged 1 commit intoprojectdiscovery:mainfrom
iamleot:permission_linux_only_linux

Conversation

@iamleot
Copy link
Copy Markdown
Contributor

@iamleot iamleot commented Feb 26, 2023

Given the filename with the _GOOS suffix this file can be build only on linux (most other Unix-es should be already handled via permission_{darwin,linux}.go).

NFCI.

Given the filename with the _GOOS suffix this file can be build only on
linux (most other Unix-es should be already handled via
permission_{darwin,linux}.go).

NFCI.
@iamleot iamleot marked this pull request as ready for review February 26, 2023 16:15
@ehsandeep ehsandeep requested a review from Mzack9999 February 26, 2023 18:21
@iamleot
Copy link
Copy Markdown
Contributor Author

iamleot commented Feb 26, 2023

To elaborate a bit further... according go help buildconstraints the *_GOOS files can already be built only on that GOOS so the || unix will be never satisfied on non-Linux unix OSes.

Apart that the file already have Linuxisms so is not suitable to any other Unix-es.

(Noticed while inspecting older version of this code due build failures on NetBSD)

@Mzack9999 Mzack9999 merged commit 1088141 into projectdiscovery:main Feb 26, 2023
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