Skip to content
This repository has been archived by the owner on Dec 19, 2017. It is now read-only.

polymer docs aren't building #255

Closed
DartBot opened this issue Jun 5, 2015 · 16 comments
Closed

polymer docs aren't building #255

DartBot opened this issue Jun 5, 2015 · 16 comments

Comments

@DartBot
Copy link

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/5479?v=3" align="left" width="96" height="96"hspace="10"> Issue by sethladd
Originally opened as dart-lang/sdk#20285


From the logs:

http://www.dartdocs.org/buildlogs/b-f628d1cb5467cb1d7ec8096ed8e95-startupscript.log

Jul 30 21:49:15 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: Resolving dependencies...
Jul 30 21:49:15 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript:
Jul 30 21:49:15 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: Incompatible version constraints on barback:
Jul 30 21:49:15 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: - polymer 0.12.0+1 depends on version >=0.14.2 <0.15.0
Jul 30 21:49:15 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: - pub itself depends on version >=0.13.0 <0.14.2
Jul 30 21:49:15 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript:
Jul 30 21:49:15 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: workingDirectory = /tmp/build_documentation_cache/hosted/pub.dartlang.org/polymer-0.12.0+1
Jul 30 21:49:15 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: docgen [--compile, --no-include-sdk, --no-include-dependent-packages, --out, docs, --sdk, /dart-sdk, --package-root, ./packages, .]
Jul 30 21:49:17 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: Using default options for documenting a single package: --start-page=polymer
Jul 30 21:49:17 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: Package Root: ./packages
Jul 30 21:49:17 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: Added to libraries: ./lib/auto_binding.dart
Jul 30 21:49:17 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: Added to libraries: ./lib/transformer.dart
Jul 30 21:49:17 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: Added to libraries: ./lib/polymer.dart
Jul 30 21:49:17 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: Added to libraries: ./lib/deploy.dart
Jul 30 21:49:17 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: Added to libraries: ./lib/deserialize.dart
Jul 30 21:49:17 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: Added to libraries: ./lib/init.dart
Jul 30 21:49:17 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: Added to libraries: ./lib/builder.dart
Jul 30 21:49:17 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: SDK Root: /dart-sdk
Jul 30 21:49:17 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: lib/auto_binding.dart:8:8:
Jul 30 21:49:17 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: Error: Can't read 'package:polymer/polymer.dart' (Error reading 'packages/polymer/polymer.dart' (OS Error: No such file or directory, errno = 2)).
Jul 30 21:49:17 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: import 'package:polymer/polymer.dart';
Jul 30 21:49:17 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Jul 30 21:49:17 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: lib/auto_binding.dart:9:8:
Jul 30 21:49:17 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: Error: Can't read 'package:template_binding/template_binding.dart' (Error reading 'packages/template_binding/template_binding.dart' (OS Error: No such file or directory, errno = 2)).
Jul 30 21:49:17 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: import 'package:template_binding/template_binding.dart';
Jul 30 21:49:17 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Jul 30 21:49:17 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: lib/transformer.dart:8:8:
Jul 30 21:49:17 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: Error: Can't read 'package:barback/barback.dart' (Error reading 'packages/barback/barback.dart' (OS Error: No such file or directory, errno = 2)).
Jul 30 21:49:17 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: import 'package:barback/barback.dart';

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/5479?v=3" align="left" width="48" height="48"hspace="10"> Comment by sethladd


Ah, looks like this is the issue:

 Resolving dependencies...
Jul 30 21:49:15 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript:
Jul 30 21:49:15 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: Incompatible version constraints on barback:
Jul 30 21:49:15 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: - polymer 0.12.0+1 depends on version >=0.14.2 <0.15.0
Jul 30 21:49:15 b-f628d1cb5467cb1d7ec8096ed8e95 startupscript: - pub itself depends on version >=0.13.0 <0.14.2

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/5479?v=3" align="left" width="48" height="48"hspace="10"> Comment by sethladd


Added Duplicate label.
Marked as being merged into dart-lang/pub#1074.

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/46275?v=3" align="left" width="48" height="48"hspace="10"> Comment by munificent


Is this script using pub global activate? If not, this bug isn't a dupe.

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/5479?v=3" align="left" width="48" height="48"hspace="10"> Comment by sethladd


As far as I can tell, it's not using pub global. I'll un-duplicate.


Added Triaged label.
Marked as being merged into #.

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/5479?v=3" align="left" width="48" height="48"hspace="10"> Comment by sethladd


It's now saying this:

Jul 31 01:46:43 b-290d39c4642429eb0f77f6952d031 startupscript: Resolving dependencies...
Jul 31 01:46:43 b-290d39c4642429eb0f77f6952d031 startupscript:
Jul 31 01:46:43 b-290d39c4642429eb0f77f6952d031 startupscript: Incompatible version constraints on barback:
Jul 31 01:46:43 b-290d39c4642429eb0f77f6952d031 startupscript: - polymer 0.12.0+3 depends on version >=0.14.2 <0.16.0
Jul 31 01:46:43 b-290d39c4642429eb0f77f6952d031 startupscript: - pub itself depends on version >=0.13.0 <0.14.2


cc @nex3.
cc @sigmundch.

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/5479?v=3" align="left" width="48" height="48"hspace="10"> Comment by sethladd


I couldn't find the version of Dart that is used when building docs: http://www.dartdocs.org/buildlogs/b-290d39c4642429eb0f77f6952d031-startupscript.log Did I miss it? Looks like it's whatever is the latest dev build.

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/654526?v=3" align="left" width="48" height="48"hspace="10"> Comment by financeCoding


Seth, the version is

Jul 31 01:46:26 b-290d39c4642429eb0f77f6952d031 startupscript: + wget http://storage.googleapis.com/dart-archive/channels/dev/release/latest/sdk/dartsdk-linux-x64-release.zip -O /tmp/dartsdk-linux-x64-release.zip

I thought I added something that actually calls dart --version also. Guess didnt do that yet.

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/654526?v=3" align="left" width="48" height="48"hspace="10"> Comment by financeCoding


I guess if the polymer package was uploaded before the sdk was released then it would of failed from version out of sync.

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/654526?v=3" align="left" width="48" height="48"hspace="10"> Comment by financeCoding


Tried doing a rebuild and still failed. http://www.dartdocs.org/buildlogs/b-6ca8cfe6d4252a350e838d9f22e76-startupscript.log

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/188?v=3" align="left" width="48" height="48"hspace="10"> Comment by nex3


The most recent versions of polymer need the bleeding-edge SDK to resolve their dependencies.

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/5479?v=3" align="left" width="48" height="48"hspace="10"> Comment by sethladd


No problem. I opened adam-singer/dart-carte-du-jour#82 to ask for the logs to include the build number.

@natalie, the constraint on polymer.dart:

environment:
  sdk: '>=1.4.0 <2.0.0'

Should we make that to force 1.6-dev ?

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/188?v=3" align="left" width="48" height="48"hspace="10"> Comment by nex3


No, it's not a constraint from polymer on the SDK, but from pub on barback. The current SDKs require barback <0.14.2, but the latest polymers require barback >=0.14.2.

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/5479?v=3" align="left" width="48" height="48"hspace="10"> Comment by sethladd


Sorry, I think I misinterpreting "The most recent versions of polymer need the bleeding-edge SDK". If the most recent version of Polymer needs 1.6-dev-LATEST, why wouldn't we specify that in the SDK constraint on polymer?

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/188?v=3" align="left" width="48" height="48"hspace="10"> Comment by nex3


Because although in practice they require it to get their dependencies, they don't use any features that are specific to that version of the SDK. It effectively has a transitive dependency on that SDK.

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/5479?v=3" align="left" width="48" height="48"hspace="10"> Comment by sethladd


dartdocs.org could respect the SDK constraint in the pubspec.yaml, but if it doesn't reflect reality, how can dartdocs.org pull the correct SDK to build the docs for polymer?

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/188?v=3" align="left" width="48" height="48"hspace="10"> Comment by nex3


It does reflect reality. The code in polymer doesn't depend on anything before the 1.4.0 SDK. Pubspecs are explicitly designed not to include the full transitive set of dependencies, and this is just a special case of that design. Checking the entrypoint package's SDK constraint isn't sufficient in general for the same reason it's not sufficient here: it doesn't take into account transitive dependencies' SDK constraints.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

No branches or pull requests

2 participants