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

Add Sound Blaster Control App #83961

Closed
wants to merge 3 commits into from
Closed

Conversation

iMonZ
Copy link
Contributor

@iMonZ iMonZ commented Jun 8, 2020

Important: Do not tick a checkbox if you haven’t performed its action. Honesty is indispensable for a smooth review process.

After making all changes to a cask, verify:

Additionally, if adding a new cask:

  • Named the cask according to the token reference.
  • brew cask install {{cask_file}} worked successfully.
  • brew cask uninstall {{cask_file}} worked successfully.
  • Checked the cask was not already refused.
  • Checked the cask is submitted to the correct repo.

@vitorgalvao
Copy link
Member

Failing with

uninstall script /Applications/Creative/Uninstaller.app/Contents/MacOS/Uninstaller does not exist.

@vitorgalvao vitorgalvao added the awaiting user reply Issue needs response from a user. label Jun 8, 2020
@ran-dall
Copy link
Contributor

@iMonZ Where you going to take a look at this, or should we proceed with closing this out?

@iMonZ
Copy link
Contributor Author

iMonZ commented Jul 12, 2020 via email

@vitorgalvao
Copy link
Member

Needs to be submitted to homebrew/cask-drivers, it seems. We define drivers as:

Software to make a hardware peripheral recognisable and usable by the system. If the software is useless without the peripheral, it’s considered a driver.

Note that repo has special naming rules.

What are the names of the apps this installs under /Applications/Creative?

@iMonZ
Copy link
Contributor Author

iMonZ commented Jul 24, 2020

Needs to be submitted to homebrew/cask-drivers, it seems. We define drivers as:

Software to make a hardware peripheral recognisable and usable by the system. If the software is useless without the peripheral, it’s considered a driver.

It‘s the same kind as #83746 and this is also an Cask (because this is an control App for this device)

  • [ ]

What are the names of the apps this installs under /Applications/Creative?

This will be automatically uninstalled with the installer script!

@iMonZ
Copy link
Contributor Author

iMonZ commented Jul 24, 2020

/Applications/Creative
Creative Uninstaller.app
Product Registration.app
Sound Blaster Control Panel.app

@vitorgalvao
Copy link
Member

It‘s the same kind as #83746

Which looks like it too should be in the drivers repo.

because this is an control App for this device

If it controls a device, it definitely belongs in the other repo.

This will be automatically uninstalled with the installer script!

Not why I’m asking. I really do want the names of the apps.

@@ -0,0 +1,19 @@
cask 'sound-blaster-control' do
Copy link
Member

Choose a reason for hiding this comment

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

Would be more compliant with the rules to be sound-blaster-control-panel.

@reitermarkus
Copy link
Member

@iMonZ, please rebase this and adjust the style to the new rules.

@vitorgalvao
Copy link
Member

vitorgalvao commented Aug 30, 2020

Definitely looks like it belongs in homebrew/cask-drivers. I’ll move the other cask.

There’s been no reply on this for over a month, so I’ll close it. It can be resubmitted to the correct repo with the requested fixes.

please rebase this and adjust the style to the new rules.

User rebases remove the ability to see what changed between commits, which means we have to reevaluate the whole cask every time. Since cask submissions should always be a single commit, it’s better if we do it ourselves through the Github interface or a script (that’s an example).

@vitorgalvao vitorgalvao removed the awaiting user reply Issue needs response from a user. label Aug 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants