Skip to content
This repository has been archived by the owner on Oct 30, 2018. It is now read-only.

fix bz5472979 server resources in build #645

Merged
merged 1 commit into from
Oct 22, 2012

Conversation

isao
Copy link
Contributor

@isao isao commented Oct 18, 2012

server affinity resources do not need to be uri-
addressable. With this change store.getAllUrls()
will no longer include foo.server.js resources,
and hence be omitted from mojito build

server affinity resources do not need to be uri
addressable. with this change store.getAllUrls()
will no longer include foo.server.js resources,
and hence be omitted from `mojito build`
@isao
Copy link
Contributor Author

isao commented Oct 18, 2012

thanks @drewfish for the fast diagnosis and remedy.

smoke test:

build html5app before this PR

% tree ../aaa-build-hybrid/ | grep server
│   │   │   ├── carrier.server.js
│   │   │   ├── cookie.server.js
│   │   │   ├── deploy.server.js
│   │   │   ├── device.server.js
│   │   │   ├── http.server.js
│   │       ├── hb.server.js
│   │       └── mu.server.js
│       ├── perf.server.js
│   ├── controller.server.js
│   │   └── foo.server.js
%

after:

% tree ../aaa-build-hybrid/ | grep server
%

@drewfish
Copy link
Contributor

+1

isao pushed a commit that referenced this pull request Oct 22, 2012
fix bz 5472979 server resources in build
@isao isao merged commit 936326d into YahooArchive:develop Oct 22, 2012
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.

2 participants