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

papers: Make thumbnailer file point to absolute path #340625

Merged
merged 1 commit into from
Sep 15, 2024
Merged

papers: Make thumbnailer file point to absolute path #340625

merged 1 commit into from
Sep 15, 2024

Conversation

anewdi
Copy link
Contributor

@anewdi anewdi commented Sep 8, 2024

Description of changes

Allows use with nautilus(bubblewrap) even if installed in home.packages

Things done

  • Built on platform(s)

    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested basic functionality of all binary files (usually in ./result/bin/)

  • Fits CONTRIBUTING.md.

Copy link
Member

@tomodachi94 tomodachi94 left a comment

Choose a reason for hiding this comment

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

Please see #340624 (review).

@anewdi anewdi changed the title Make thumbnailer file point to absolute path papers: Make thumbnailer file point to absolute path Sep 11, 2024
Allows use with nautilus(bubblewrap) even if installed in home.packages
Copy link
Member

@tomodachi94 tomodachi94 left a comment

Choose a reason for hiding this comment

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

Lgtm, thank you. 💖

Would you like this backported?

@anewdi
Copy link
Contributor Author

anewdi commented Sep 11, 2024

Yeah, that would be great!

@wegank wegank added the 12.approvals: 1 This PR was reviewed and approved by one reputable person label Sep 11, 2024
@anewdi
Copy link
Contributor Author

anewdi commented Sep 15, 2024

Is there anything missing here or does it usually take some time for it to get merged?

@jtojnar jtojnar merged commit 6c8bdaa into NixOS:master Sep 15, 2024
28 checks passed
@jtojnar
Copy link
Member

jtojnar commented Sep 15, 2024

Sorry, was waiting for CI and then forgotten about it.

Copy link
Contributor

Backport failed for release-24.05, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin release-24.05
git worktree add -d .worktree/backport-340625-to-release-24.05 origin/release-24.05
cd .worktree/backport-340625-to-release-24.05
git switch --create backport-340625-to-release-24.05
git cherry-pick -x 8ce371c4840c624f49796e32913287ae94092038

@anewdi
Copy link
Contributor Author

anewdi commented Sep 15, 2024

I tried to follow the steps from the bot, but what do i do now? Where do i push?

@anewdi
Copy link
Contributor Author

anewdi commented Sep 15, 2024

Manual pull request?

@jtojnar
Copy link
Member

jtojnar commented Sep 15, 2024

Yes, manual pull request will be needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants