Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix cuttlefish #8

Merged
merged 4 commits into from
Mar 2, 2016
Merged

Fix cuttlefish #8

merged 4 commits into from
Mar 2, 2016

Conversation

davecromberge
Copy link
Contributor

Same issue as vernemq/rebar3_cuttlefish#7, upgrade rebar3 to newer version as suggested.

@@ -9,7 +9,7 @@
{deps,
[
lager,
{cuttlefish, ".*", {git, "https://github.com/tsloughter/cuttlefish.git", {branch, "develop"}}},
{cuttlefish, "2.0.7"},
Copy link
Contributor

Choose a reason for hiding this comment

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

this is not needed any more, it shouldn't be included or it can conflict with the cuttlefish plugin

@Licenser
Copy link
Contributor

Licenser commented Mar 2, 2016

Hi thanks for this I haven't gotten around to updating the FE yet, sorry.

A few points:

  1. (see inline comment)
  2. {copy, "_build/default/bin/cuttlefish", "bin/cuttlefish"}, needs to be gone from the relx section
  3. the schema sources should be prod not default as a base

@davecromberge
Copy link
Contributor Author

@Licenser thanks for the feedback, I have addressed all but the last point. Can you give a slightly more detailed description?

@Licenser
Copy link
Contributor

Licenser commented Mar 2, 2016

Awesome thanks!

the {template, "_build/default/lib/dqe/priv/dqe.schema", "share/schema/01-dqe.schema"} needs to read
{template, "_build/prod/lib/dqe/priv/dqe.schema", "share/schema/01-dqe.schema"} since when build with prod it might not be available in default.

@davecromberge
Copy link
Contributor Author

Do you mean to use \{\{build_dir\}\}/lib/ instead?

@Licenser
Copy link
Contributor

Licenser commented Mar 2, 2016

not sure if that gets substituted, does it?

@davecromberge
Copy link
Contributor Author

@Licenser it's not something automatic, needed some extra config in vars.config

@Licenser
Copy link
Contributor

Licenser commented Mar 2, 2016

they don't apply top the path I think, it only applies to the files.

But what I mean is it needs to read:
{template, "_build/prod/lib/dqe/priv/dqe.schema", "share/schema/01-dqe.schema"}
not
{template, "_build/default/lib/dqe/priv/dqe.schema", "share/schema/01-dqe.schema"}

@davecromberge
Copy link
Contributor Author

I made the change as you suggest, but it only works when I do a rebar3 as prod release, in the default profile it no longer would work. So in order to get the release to work under both profiles, I introduced the build_path config, which detects the profile. I may be mistaken, because the rebar3 stuff is still a bit cryptic at this point to me.

@Licenser
Copy link
Contributor

Licenser commented Mar 2, 2016

awesome, and wow that's icky but I see why it's needed.

Licenser added a commit that referenced this pull request Mar 2, 2016
@Licenser Licenser merged commit ddb1bac into dalmatinerdb:test Mar 2, 2016
@davecromberge
Copy link
Contributor Author

Sure @Licenser I agree, it is icky, I can't take the credit though as my colleague found it out when our CI stopped working for dalmatinerdb.

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.

None yet

2 participants