Adding in v2.0 Beta option for Alfred #2

Closed
wants to merge 3 commits into
from

Projects

None yet

5 participants

@pyro2927

v2.0 beta is available to Mega Supporters. Updating module (recipe? pattern?) to allow for

include alfred::v2beta

v2.0 is quite different than v1, which I think earns it an honest 'version bump'.

@wfarr wfarr commented on the diff Feb 27, 2013
manifests/v2beta.pp
@@ -0,0 +1,11 @@
+# Public: Install Alfred.app into /Applications.
+#
+# Examples
+#
+# include alfred
+class alfred::v2beta {
@wfarr
wfarr Feb 27, 2013 Member

This class should inherit from alfred and override the source so only one can be included.

@wfarr
Member
wfarr commented Mar 5, 2013

Bump.

@dgoodlad
Member

I'd actually put forward that Alfred 2 deserves it's own module, like with istatmenus{3,4}. It's possible to have both apps live side-by-side, since they're named Alfred.app and Alfred 2.app respectively. They also maintain separate preferences as far as I can tell. Some people may even want to have both around, as crazy as it sounds.

Thoughts?

@wfarr
Member
wfarr commented Mar 14, 2013

Sounds legit to me it they can be installed side by side

Will Farrington
System Operations Engineer
GitHub, Inc.

On Mar 14, 2013, at 7:46 AM, David Goodlad notifications@github.com wrote:

I'd actually put forward that Alfred 2 deserves it's own module, like with istatmenus{3,4}. It's possible to have both apps live side-by-side, since they're named Alfred.app and Alfred 2.app respectively. They also maintain separate preferences as far as I can tell. Some people may even want to have both around, as crazy as it sounds.

Thoughts?


Reply to this email directly or view it on GitHub.

@dgoodlad
Member

I've created a module at dgoodlad/puppet-alfred2 which should do the trick.

@pyro2927 Do you think this is a suitable replacement for what you were trying to do with this PR?

@dgoodlad dgoodlad referenced this pull request in boxen/our-boxen Mar 14, 2013
Closed

puppet-alfred2 #177

@geetarista
Contributor

I don't see the point for having a separate repository. Many Mac applications can be installed side by side as different versions, but it isn't common to actually do so. If someone wants to use an older version, they should just be able to specify it when calling the class. Same goes for having an alfred::beta class.

Anyway, that's just me. 😄

@skalnik skalnik referenced this pull request Mar 15, 2013
Merged

update Alfred to 2.0 #4

@fromonesrc
Member

Closing because #4.

@fromonesrc fromonesrc closed this Apr 5, 2013
@polymetis polymetis added a commit to polymetis/puppet-alfred that referenced this pull request Apr 18, 2013
@polymetis polymetis bumps version - fixes boxen/puppet-alfred#2 fd7055d
@polymetis polymetis added a commit to polymetis/puppet-alfred that referenced this pull request Apr 18, 2013
@polymetis polymetis bumps version - fixes boxen/puppet-alfred#2 db233fd
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment