Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

apous v0.2.1 #41473

Closed
wants to merge 2 commits into from
Closed

apous v0.2.1 #41473

wants to merge 2 commits into from

Conversation

owensd
Copy link

@owensd owensd commented Jul 7, 2015

No description provided.

url "https://github.com/owensd/apous/archive/v0.2.1.tar.gz"
sha256 "d2a283bd328c7dae117de6078448e87a3f3b960876608dd238f5b87f698a10bc"

depends_on :xcode => ["7.0", :build]
Copy link
Member

Choose a reason for hiding this comment

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

Just to double check: You can remove xcode from your system after installing this and it'll continue to work as expected?

Copy link
Author

Choose a reason for hiding this comment

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

Yes, Xcode is required for building only.

Sent from my iPhone

On Jul 7, 2015, at 6:13 PM, Dominyk Tiller notifications@github.com wrote:

In Library/Formula/apous.rb:

@@ -0,0 +1,23 @@
+class Apous < Formula

  • depends_on :xcode => ["7.0", :build]
    Just to double check: You can remove xcode from your system after installing this and it'll continue to work as expected?


Reply to this email directly or view it on GitHub.

Copy link
Author

Choose a reason for hiding this comment

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

Sorry, to be clear. You still need the xcodebuild command line utilities for it to work.

Sent from my iPhone

On Jul 7, 2015, at 6:13 PM, Dominyk Tiller notifications@github.com wrote:

In Library/Formula/apous.rb:

@@ -0,0 +1,23 @@
+class Apous < Formula

  • depends_on :xcode => ["7.0", :build]
    Just to double check: You can remove xcode from your system after installing this and it'll continue to work as expected?


Reply to this email directly or view it on GitHub.

Copy link
Member

Choose a reason for hiding this comment

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

If a user is running Homebrew it's safe to assume they'll always have either Xcode or the CLT installed, so as long as it can run with the CLT installed it's fine for the dep to be :build.

Copy link
Member

Choose a reason for hiding this comment

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

Is this a stable version of Xcode on a stable version of OS X?

Copy link
Member

Choose a reason for hiding this comment

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

Heh, well spotted. 7.0 is still beta. You can run it on 10.10 though, if you super want to.

Copy link
Member

Choose a reason for hiding this comment

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

Will probably need to hold off on this until 7.0 is stable.

Copy link
Author

Choose a reason for hiding this comment

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

Ok, thanks. I can resubmit when public.

On Jul 8, 2015, at 9:14 AM, Mike McQuaid notifications@github.com wrote:

In Library/Formula/apous.rb #41473 (comment):

@@ -0,0 +1,23 @@
+class Apous < Formula

  • depends_on :xcode => ["7.0", :build]
    Will probably need to hold off on this until 7.0 is stable.


Reply to this email directly or view it on GitHub https://github.com/Homebrew/homebrew/pull/41473/files#r34166812.

@Danappelxx
Copy link

What about now? Xcode 7 is both stable and the version you can download from the app store.

@DomT4
Copy link
Member

DomT4 commented Oct 7, 2015

@BrewTestBot test this please

@DomT4
Copy link
Member

DomT4 commented Oct 12, 2015

Currently not building against Xcode 7.0.1:

/tmp/apous20151006-61864-l8au80/apous-0.2.1/src/Tools.swift:166:46: error: 'stringByAppendingPathComponent' is unavailable: Use URLByAppendingPathComponent on NSURL instead.
        if fileManager.fileExistsAtPath(path.stringByAppendingPathComponent(CartfileConfig)) {
                                             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Foundation.String:352:8: note: 'stringByAppendingPathComponent' has been explicitly marked unavailable here
  func stringByAppendingPathComponent(aString: String) -> String
       ^
/tmp/apous20151006-61864-l8au80/apous-0.2.1/src/Tools.swift:171:46: error: 'stringByAppendingPathComponent' is unavailable: Use URLByAppendingPathComponent on NSURL instead.
        if fileManager.fileExistsAtPath(path.stringByAppendingPathComponent(PodfileConfig)) {
                                             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Foundation.String:352:8: note: 'stringByAppendingPathComponent' has been explicitly marked unavailable here
  func stringByAppendingPathComponent(aString: String) -> String
       ^
/tmp/apous20151006-61864-l8au80/apous-0.2.1/src/Tools.swift:77:54: error: 'advance' is unavailable: call the 'advancedBy(n)' method on the index
            string.replaceRange(range.startIndex ..< advance(range.startIndex, 2), with: "\\e")
                                                     ^~~~~~~
Swift.advance:1:89: note: 'advance' has been explicitly marked unavailable here
@available(*, unavailable, message="call the 'advancedBy(n)' method on the index") func advance<T : ForwardIndexType>(start: T, _ n: T.Distance) -> T
                                                                                        ^

** INSTALL FAILED **


The following build commands failed:
    CompileSwift normal x86_64 /tmp/apous20151006-61864-l8au80/apous-0.2.1/src/Utils.swift
    CompileSwift normal x86_64 /tmp/apous20151006-61864-l8au80/apous-0.2.1/src/main.swift
    CompileSwift normal x86_64 /tmp/apous20151006-61864-l8au80/apous-0.2.1/src/Tools.swift
    CompileSwiftSources normal x86_64 com.apple.xcode.tools.swift.compiler
(4 failures)

@bfontaine
Copy link
Contributor

Ping?

@owensd
Copy link
Author

owensd commented Oct 19, 2015

Version 0.2.2 has fixed this issue: https://github.com/owensd/apous/releases/tag/v0.2.2. Should I update the pull and made a new request?

@bfontaine
Copy link
Contributor

@owensd Yes you can update this PR.

@DomT4
Copy link
Member

DomT4 commented Dec 12, 2015

Closing as dead. Please feel free to resubmit a fresh PR as desired. Thanks for your submission to Homebrew!

@DomT4 DomT4 closed this Dec 12, 2015
@Homebrew Homebrew locked and limited conversation to collaborators Jul 10, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants