Skip to content

papis: enable check.#61003

Open
orahcio wants to merge 2 commits into
void-linux:masterfrom
orahcio:fix-papis-habanero
Open

papis: enable check.#61003
orahcio wants to merge 2 commits into
void-linux:masterfrom
orahcio:fix-papis-habanero

Conversation

@orahcio

@orahcio orahcio commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Testing the changes

  • I tested the changes in this PR: YES

Local build testing

  • I built this PR locally for my native architecture, (x86-64)
  • I built this PR locally for these architectures (if supported. mark crossbuilds):
    • i686

Papis package had checks disabled because one failed test. Because this the command papis add --from doi DOI file.pdf could not collect information form doi service. I think, this behavior occurred because papis uses actually a old habanero version than the void one.

@orahcio

orahcio commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

Could someone re-run the job? It not seems a fail because the template or the patch.

@classabbyamp

Copy link
Copy Markdown
Member

just force-push. this should be one commit anyways

@orahcio orahcio force-pushed the fix-papis-habanero branch from 53a4e3f to b6b6e4e Compare June 11, 2026 17:14
@classabbyamp

Copy link
Copy Markdown
Member

squash this please

@orahcio

orahcio commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

Ok, I am out now. But I could see that patch makes the test to pass, but the runtime error is the same. I will try to fix it with the actual habanero version on void.

Actually papis runs almost perfectly, just the function to add info from doi service fails.

@orahcio orahcio force-pushed the fix-papis-habanero branch from b6b6e4e to 028b410 Compare June 11, 2026 22:20
@orahcio

orahcio commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

Now, papis works perfectly. I could see that the problem is a very old habanero library, not papis using a old habanero version, maybe they need to fix the habanero version condition to >1 in their pyproject.toml. But I think that library needs a update here. My next PR will be to update habanero.

@orahcio orahcio force-pushed the fix-papis-habanero branch 4 times, most recently from b379ba3 to be1567d Compare June 12, 2026 02:24
@orahcio orahcio changed the title papis: fix crossref.py papis: enable check. Jun 12, 2026
@orahcio orahcio force-pushed the fix-papis-habanero branch 5 times, most recently from 0474f87 to f364dc0 Compare June 12, 2026 03:33
@orahcio

orahcio commented Jun 12, 2026

Copy link
Copy Markdown
Contributor Author

I changed this PR to remove papis patches. The package works very well with python3-habanero-2.3.0. I promise not to change this PR anymore, except for reviews.

Comment thread srcpkgs/python3-habanero/template Outdated
@orahcio orahcio force-pushed the fix-papis-habanero branch from f364dc0 to a30b119 Compare June 12, 2026 23:22
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.

3 participants