OkHttp versions 5.4.0+ no longer include baseline-prof.txt
Maintainers usually reply within 1 day
Assessment
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Newbie friendliness
- 83/100
Research direction
The issue identifies okhttp/src/androidMain/baseline-prof.txt and the expected src/androidMain/baselineProfiles/ directory. Check how the Android Kotlin Multiplatform library plugin packages baseline profiles, then move the file to the expected location and inspect a built okhttp-android AAR for baseline-prof.txt.
Written by the indexing model from the issue text.
Description
Context
From OkHttp 5.4.0+, the okhttp-android AAR no longer includes baseline-prof.txt. The cause is https://github.com/lysine-dev/okhttp/pull/9269, which moved OkHttp to the new com.android.kotlin.multiplatform.library plugin.
Repro
If you download the 5.3.2 version you'll see baseline-prof.txt is packaged in the AAR. 5.4.0, 5.5.0 and 5.5.1 have none.
Quick script to download those versions and check for the baseline profile:
mkdir -p /tmp/okhttp-aar && cd /tmp/okhttp-aar
for v in 5.3.2 5.4.0 5.5.1; do
curl -sO https://repo1.maven.org/maven2/com/squareup/okhttp3/okhttp-android/$v/okhttp-android-$v.aar
echo "$v:"; unzip -l okhttp-android-$v.aar | grep baseline-prof || echo " missing"
done
Suggested Fix
The file is still at okhttp/src/androidMain/baseline-prof.txt, but the KMP library plugin only reads profiles from src/androidMain/baselineProfiles/ so nothing packages it. Just need to move it to that folder dir and it should work.
- Dominant language
- Kotlin
- Stars
- 47.1k
- Forks
- 9.3k
- Avg merge
- 1d 6h
- Merged PRs (30d)
- 29
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from lysine-dev/okhttp
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 67/100
lysine-dev/okhttp#9817 ·
Maintainers usually reply within 1 day
-
Cannot add Content-Length header to a multipart partPossibly taken @Amir-Golmoradi claimed this 9 days ago. Openbug
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
lysine-dev/okhttp#9814 · 1 comment · 1 reaction ·
Maintainers usually reply within 1 day
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
lysine-dev/okhttp#9761 · 3 comments ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 30/100
lysine-dev/okhttp#9838 ·
Maintainers usually reply within 1 day
-
bug
Difficulty 4/5 3-5 days Newbie friendliness 48/100
lysine-dev/okhttp#9821 · 3 comments ·
Maintainers usually reply within 1 day
All issues in lysine-dev/okhttp
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 80/100
open-telemetry/opentelemetry-android#2183 ·
Maintainers usually reply within 2 days
-
Difficulty 1/5 Under an hour Newbie friendliness 62/100
Droid-Deck/DroidDeck#2058 ·
Maintainers usually reply within 1 day
-
bug core p3
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
RunanywhereAI/runanywhere-sdks#1258 ·
Maintainers usually reply within 1 day
-
bug
Difficulty 1/5 Under an hour Newbie friendliness 78/100
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
axelixlabs/axelix#1918 ·
Maintainers usually reply within 1 day