这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@michaelshobbs
Copy link
Member

i'm not sure this is the best solution but it works in leiu of making the plugins handle it

closes #1609

@mmerickel
Copy link
Contributor

Is --link the only option expected to be used by plugins at this stage not used by herokuish/buildstep? My first impression is that this fix is short-sighted but I'm also suffering from the issue so I'm happy if this works as a temporary fix.

What about dockerfile plugins that don't work for herokuish builds like --build-arg or --file?

@michaelshobbs
Copy link
Member Author

Probably not and it is short-sighted indeed. However, leaving this broken is probably not good either.

EDIT: Also I don't follow what plugins you're referring to when you reference --build-arg or --file. Do you mean plugins that add those options?

@mmerickel
Copy link
Contributor

Yeah I was just bringing up the inverse problem where a plugin is expecting a dockerfile deployment.

It does seem beneficial to tell the plugin which deployment type is happening to really solve this issue.

@michaelshobbs michaelshobbs force-pushed the 1609_filter-link-with-dockerfile branch from cfe967b to d0032f2 Compare November 25, 2015 01:13
@michaelshobbs
Copy link
Member Author

Updated to take into account the inverse problem. This is by no means exhaustive but gets us a pattern to filter args for now.

@michaelshobbs michaelshobbs changed the title filter --link docker option when building dockerfile app filter incompatible docker option when building dockerfile vs herokuish apps Nov 25, 2015
@josegonzalez
Copy link
Member

@michaelshobbs you want to add that static buildpack and a test app to herokuish?

@michaelshobbs
Copy link
Member Author

I can yeah. My fork or the official one?

@josegonzalez
Copy link
Member

Your fork. Or just move your fork to the dokku org and be done with it. Official fork?

@michaelshobbs
Copy link
Member Author

Yeah I'll revert this change here and open a new PR

@michaelshobbs michaelshobbs force-pushed the 1609_filter-link-with-dockerfile branch from f512120 to d0032f2 Compare November 25, 2015 17:05
@michaelshobbs michaelshobbs force-pushed the 1609_filter-link-with-dockerfile branch from d0032f2 to 2c34a60 Compare November 25, 2015 17:09
michaelshobbs added a commit that referenced this pull request Nov 25, 2015
filter incompatible docker option when building dockerfile vs herokuish apps
@michaelshobbs michaelshobbs merged commit 6495537 into master Nov 25, 2015
@michaelshobbs michaelshobbs deleted the 1609_filter-link-with-dockerfile branch November 25, 2015 19:14
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.

Cannot run first build using docker and postgres

4 participants