E524 Brands renaming by Simbiat · Pull Request #8109 · matomo-org/device-detector · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@Simbiat
Copy link
Contributor
@Simbiat Simbiat commented Sep 21, 2025

Description:

Split from #8096 this covers brand name changes, that are mostly more than just addition of a prefix or otherwise more contentious

Review

@liviuconcioiu
Copy link
Collaborator

@sgiehl I've reviewed the changes, all are fine for me. @sanchezzzhak if you can take a second look please?

@Simbiat Simbiat mentioned this pull request Jan 21, 2026
10 tasks
@Simbiat
Copy link
Contributor Author
Simbiat commented Jan 26, 2026

Fails on composer

dyld[5847]: Library not loaded: @loader_path/../../../../opt/jpeg/lib/libjpeg.9.dylib
  Referenced from: <2E6E9A91-A871-384A-A669-F914E7D3FB65> /opt/homebrew/Cellar/php@7.2/7.2.34_15/bin/php
  Reason: tried: '/opt/homebrew/Cellar/php@7.2/7.2.34_15/bin/../../../../opt/jpeg/lib/libjpeg.9.dylib' (no such file)
/Users/runner/work/_temp/bc4ad433-2388-4fc1-9862-96d78e1273aa.sh: line 1:  5847 Abort trap: 6           composer install

don't think this was caused by me 😅

- regex: 'Cherry(?:[ _]?Mobile)?[ _]?([^/;]+)(?: Build|[);])'
model: '$1'

#Previously listed as "White Mobile" brand, but based on https://firmwarefile.com/cherry-mobile-omega-lite-3, https://forum.gsmhosting.com/vbb/f296/cherry-mobile-omega-lite-3-frp-remove-2259709/ and https://deviceatlas.com/device-data/devices/white-mobile/am1010v/30376111 (and others), this is likely "Cherry Mobile", especially since I was not able to find any evidence of "White Mobile" ever existing.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove this comment

Cherry Mobile should stay with the old name.
https://www.gadgetpilipinas.net/2016/12/cherry-mobile-defender/

The AM1010V can be separated into a separate brand.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

But I did not change name for Cherry Mobile. What do you mean?
Are you suggesting we keep White Mobile for AM1010V?

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

0