Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Go package clean up #26377

Merged
merged 1 commit into from
Apr 18, 2021
Merged

Go package clean up #26377

merged 1 commit into from
Apr 18, 2021

Conversation

ericonr
Copy link
Member

@ericonr ericonr commented Nov 14, 2020

While I get ready to do a full go rebuild, I found packages with various issues. Some broken ones and some had changed paths.

ping @the-maldridge @thypon @Vaelatern @misuchiru03 @renatoaguiar @steinex @CameronNemo

@ericonr ericonr force-pushed the gorebuild branch 4 times, most recently from 8a887c4 to fc046c5 Compare November 14, 2020 19:25
@pullmoll
Copy link
Member

For chronograf see #26197

@@ -18,6 +18,7 @@ system_accounts="_chronograf"
_chronograf_homedir="/var/lib/${pkgname}"
make_dirs="${_chronograf_homedir} 0755 _chronograf _chronograf"

broken="fails to build v8: error: 'class Nan::Persistent<v8::Object>' has no member named 'IsNearDeath'"
Copy link
Member

Choose a reason for hiding this comment

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

Might be worth just switching to upstream assets if they vendor a tarball.

Copy link
Member Author

Choose a reason for hiding this comment

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

Not sure this one is necessary anymore, since you merged the fix.

@@ -11,3 +11,4 @@ license="GPL-3.0-or-later"
homepage="https://github.com/snail007/goproxy"
distfiles="https://github.com/snail007/goproxy/archive/v${version}.tar.gz"
checksum=13f8e37d274d2b7b7cdf1ae3d98723fbb9acf1476c92a1e0d43039109d238ccc
broken="7.0 tag was removed from repository"
Copy link
Member

Choose a reason for hiding this comment

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

Reviewing the upstream licensing this whole project appears kind of shady. There is a v10.1 tag, is there a reason you did not use that instead of upgrading?

Copy link
Member Author

Choose a reason for hiding this comment

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

By upgrading do you mean marking as broken?

I didn't want to update the package without having an idea of how it works / is supposed to work, and using a newer tag doesn't protect us from them dropping it in the future again. I'd consider dropping the package, if you think it makes sense.

wrksrc="cli-${version}"
wrksrc="dnote-${version}"
Copy link
Member

@sgn sgn Nov 15, 2020

Choose a reason for hiding this comment

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

I would say this is worth to warrant a rebuild.
Thinking a bit more, there's something that was changed here. The distfiles is still the same but the extracted directory is different. Have you check if they have same source?

Comment on lines 3 to 5
version=0.2
revision=1
wrksrc="gfuzz-${version}"
wrksrc="gfz-${version}"
Copy link
Member

Choose a reason for hiding this comment

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

Same comment with dnote-cli

@@ -2,6 +2,7 @@
pkgname=miniflux
version=2.0.24
revision=1
wrksrc=v2-${version}
Copy link
Member

Choose a reason for hiding this comment

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

We need to diff the distfiles, and warrant a rebuild.

@ericonr
Copy link
Member Author

ericonr commented Nov 15, 2020

Thanks, I will investigate.

@the-maldridge
Copy link
Member

@ericonr what's the status on this PR?

ericonr referenced this pull request Jan 10, 2021
Setting LDFLAGS="-fuse-ld=bfd" does no longer work. However all
targets except ppc64le* support using the internal linker instead.

I found no way to make the ppc64le* build work and left it
trying to use the old method.
@ericonr ericonr merged commit 18ef8e1 into void-linux:master Apr 18, 2021
@ericonr ericonr deleted the gorebuild branch April 18, 2021 21:05
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants