Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

new cask for manager of mi-wifi #4444

Merged
merged 1 commit into from
May 23, 2014
Merged

new cask for manager of mi-wifi #4444

merged 1 commit into from
May 23, 2014

Conversation

y00rb
Copy link
Contributor

@y00rb y00rb commented May 17, 2014

mi-wifi is one network router of wireless. it has one batch client management tools. this cask is manager in mac version

@vitorgalvao
Copy link
Member

As per the naming reference, the file should be named xmrouter.rb with class Xmrouter.

This does seem like a good candidate for an exception, though, so let’s figure out if it is. Do you know why is the app called XMRouter.app? We can rename apps in homebrew-cask; do you think it’d be appropriate in this case?

@rolandwalker
Copy link
Contributor

I am definitely in favor of following the upstream App name wherever possible.

With Bitcoin-core, there was a weird situation where the application menu read differently than the bundle filename. We renamed using :target to resolve a contradiction and reduce confusion.

@vitorgalvao
Copy link
Member

Agreed. Still, in this case, and looking into it a bit more, the app name we actually see in finder is 小米路由器, not XMRouter.app. It’s somewhat similar to DEVONthink Pro Office, in this regard (except in this case there isn’t also a possible conflict).

@rolandwalker
Copy link
Contributor

I tried to investigate the plist file, but could not complete the download.

@vitorgalvao
Copy link
Member

Here’s the plist’s text, and here’s a mirror of the dmg (both temporary). It is referred to as “XMRouter”; even the bundle identifier is com.xiaomi.xmrouter.

@rolandwalker
Copy link
Contributor

Downloaded, thanks.

The bundle on disk is called "XMRouter.app", which is what you will see in ls.

The reason it displays differently in Finder is that the English localization strings in XMRouter.app/Contents/Resources/en.lproj/InfoPlist.strings are in Chinese. (You are seeing the value of the CFBundleDisplayName key.) This is not the only app that uses foreign languages specifically in the English locale -- it seems contrary to the purpose of having a locale, but there must be some reason developers do this.

We have a number of Chinese-language apps now, which is a good thing. I wish we had a consistent way of naming them, but we currently don't.

@y00rb
Copy link
Contributor Author

y00rb commented May 22, 2014

XM is short name Xiao mi(小米), router is application to manage wireless router.

@y00rb
Copy link
Contributor Author

y00rb commented May 22, 2014

sorry, busy in these days, will rename it asap.

@y00rb
Copy link
Contributor Author

y00rb commented May 22, 2014

so I will rename it to xmrouter.rb.
btw I have use English in my mac too, installs & works in my mac. how was your problem in different language issue?

@rolandwalker
Copy link
Contributor

OK @yangbodotnet, I think xmrouter.rb will be fine.

We didn't have a error here. We were just looking at how Finder represented the name, and continuing a long-running conversation about how we make names consistent.

@y00rb
Copy link
Contributor Author

y00rb commented May 22, 2014

@rolandwalker changed :)
looking & learning your conversation too

sha256 'f56e02b26e1398fc816e1cea28d1a5ffdcde79c47c6d16d14b0efab67e23d86d'
link 'XMRouter.app'
end

Copy link
Member

Choose a reason for hiding this comment

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

Could you please remove this extra line?

@y00rb
Copy link
Contributor Author

y00rb commented May 23, 2014

@vitorgalvao thank you very much, your comment help me found other issues, my cask is wrong location. all of them has been fixed.

@vitorgalvao
Copy link
Member

Thank you for the contribution and your work on this. Merged.

vitorgalvao added a commit that referenced this pull request May 23, 2014
new cask for manager of mi-wifi
@vitorgalvao vitorgalvao merged commit 0e5d15d into Homebrew:master May 23, 2014
@Homebrew Homebrew locked and limited conversation to collaborators May 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants