Skip to content

conmon: remove unnecessary dependency#17204

Merged
neheb merged 1 commit intoopenwrt:masterfrom
oskarirauta:conmon
Nov 29, 2021
Merged

conmon: remove unnecessary dependency#17204
neheb merged 1 commit intoopenwrt:masterfrom
oskarirauta:conmon

Conversation

@oskarirauta
Copy link
Contributor

@oskarirauta oskarirauta commented Nov 25, 2021

this patch removes unnecessary dependency to golang/host
as pointed out by @jefferyto earlier at #17063

Signed-off-by: Oskari Rauta oskari.rauta@gmail.com

Maintainer: Oskari Rauta / @oskarirauta
Compile tested: x86_64, git
Run tested: x86_64, git

@BKPepe
Copy link
Member

BKPepe commented Nov 25, 2021

  • dependency

Otherwise, this comment: #17063 (comment) was more clear than your commit description, but from my side, it is nitpicking.

@oskarirauta
Copy link
Contributor Author

  • dependency

Otherwise, this comment: #17063 (comment) was more clear than your commit description, but from my side, it is nitpicking.

I agree.

@oskarirauta
Copy link
Contributor Author

@BKPepe how about approving it then?

@oskarirauta oskarirauta changed the title conmon: remove unnecessary depency conmon: remove unnecessary dependency Nov 25, 2021
this patch removes unnecessary dependency to golang/host
as pointed out by @jefferyto earlier at openwrt#17063 (comment)

Signed-off-by: Oskari Rauta <oskari.rauta@gmail.com>
@BKPepe BKPepe requested a review from jefferyto November 26, 2021 10:22
@neheb neheb merged commit 537b6bd into openwrt:master Nov 29, 2021
@oskarirauta oskarirauta deleted the conmon branch November 29, 2021 21:31
@jefferyto
Copy link
Member

Apologies for not reviewing this PR in time.

@oskarirauta In the future, I suggest adding the actual comment link (https://github.com/openwrt/packages/pull/17063#issuecomment-962192883) in the commit message instead of the text #17063 (comment), since that text does not link to the actual comment.

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.

4 participants