MetaMask / metamask-mobile

Mobile web browser providing access to websites that use the Ethereum blockchain
https://metamask.io
Other
2.12k stars 1.1k forks source link

Duplicate metamask font on Android assets. #3429

Open giomurru opened 2 years ago

giomurru commented 2 years ago

Description

Duplicate metamask font on Android assets.

Technical Details

When cloning the repo I get this warning:

warning: the following paths have collided (e.g. case-sensitive paths
on a case-insensitive filesystem) and only one from the same
colliding group is in the working tree:

  'android/app/src/main/assets/fonts/Metamask.ttf'
  'android/app/src/main/assets/fonts/metamask.ttf'

I checked the md5 of the two files and it looks like they are the same file.

I think a possible solution can be to delete one of the ttf. I suggest to keep the one starting with the capital letter just to stay aligned with the style of the other fonts contained in the same folder.

Before deleting the duplicate font file you should verify any case-sensitive dependence in the android app code.

elliotsayes commented 2 years ago

How has this issue been open for nearly a year?