main/linux-postmarketos-qcom: rename to linux-postmarketos-qcom-msm8974
I mentioned this in https://gitlab.com/postmarketOS/pmaports/merge_requests/582 already, and there is justification for this change in the commit message and in https://gitlab.com/postmarketOS/pmaports/merge_requests/582#implementation ("Implementation").
From https://gitlab.com/postmarketOS/pmaports/merge_requests/582#note_210729198:
(
linux-postmarketos-qcom
is really more alinux-postmarketos-qcom-msm8974
at this point...)It would be a good idea to rename it then, it would be great if you could submit a follow up MR after this one that does this change.
I tested building this locally since I suspect that CI will time out.
Merge request reports
Activity
- Resolved by Administrator
Even though I don't really like having multiple packages like that, it makes much sense in how the current development is being done.
By Luca Weiss on 2019-10-05T14:30:05
Edited by Ghost User
- Resolved by Administrator
- Resolved by Administrator
- Resolved by Administrator
- Resolved by Administrator
- Resolved by Administrator
@konradybcio Although I appreciate your comments, they are in no way related to the change made in this MR. All I did is rename the existing package to give it a less confusing name. The switch from https://gitlab.com/postmarketOS/linux-postmarketos/commits/qcom to masneyb's fork was made in another MR, and all devices referencing
linux-postmarketos-qcom
now point to the masneyb fork, even without the changes in this MR.So I suggest that you open a new issue with your comments/suggestions and start discussing them separately.
By Minecrell on 2019-10-06T14:16:50
Edited by Administratoradded 22 commits
-
a387a1c2...0ff1dadc - 21 commits from branch
postmarketOS:master
- a8551648 - main/linux-postmarketos-qcom: rename to linux-postmarketos-qcom-msm8974 (!662)
By Oliver Smith on 2019-10-07T22:52:42
-
a387a1c2...0ff1dadc - 21 commits from branch
enabled an automatic merge when the pipeline for a8551648 succeeds
By Oliver Smith on 2019-10-07T22:53:05
mentioned in commit a8551648
By Minecrell on 2019-11-04T13:02:39