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

Fix broken build #99

Closed
wants to merge 1 commit into from
Closed

Fix broken build #99

wants to merge 1 commit into from

Conversation

iavael
Copy link

@iavael iavael commented Dec 15, 2022

Fixes linter error https://buildbot.flathub.org/#/builders/19/builds/1459
Fixes broken deltachat-core-rust deps deltachat/deltachat-core-rust#3747

@iavael iavael mentioned this pull request Dec 15, 2022
@muelli
Copy link
Collaborator

muelli commented Dec 15, 2022

Thanks!

hrm.
it seems I can't simply correct the typo in the commit message.

Can you fix it up to "array"?
I'd also appreciate if you could put the complaint of the bot in the commit message, so that we will know why we were actually bothered to perform the change.

Fixing "toplevel-cleanup-debug" error from flatpak-builder-lint
@flathubbot
Copy link

Started test build 11816

@flathubbot
Copy link

Build 11816 failed

@muelli
Copy link
Collaborator

muelli commented Dec 15, 2022

bot, build

@flathubbot
Copy link

Queued test build for chat.delta.desktop.

@flathubbot
Copy link

Started test build 11829

@flathubbot
Copy link

Build 11829 failed

@muelli
Copy link
Collaborator

muelli commented Dec 15, 2022

it seems to be prompting!

Username for 'https://github.com':

is there a dependency that doesn't check out well?

@iavael
Copy link
Author

iavael commented Dec 15, 2022

@muelli Yes, that's because of https://github.com/majored/rs-async-io-utilities/ repo. Looks like it was deleted.

@iavael
Copy link
Author

iavael commented Dec 15, 2022

Very interesting story here
deltachat/deltachat-core-rust#3747

Maintainer just deleted his library because he didn't need it anymore.

@iavael iavael changed the title Remove debug info to make linter happy Fix broken build Dec 15, 2022
@iavael
Copy link
Author

iavael commented Dec 15, 2022

I feel like I am duplicating work being done in #97 by @Simon-Laux right now.
Closing this PR

@iavael iavael closed this Dec 15, 2022
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