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

Maven enabled compilation #4

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Conversation

abelaska
Copy link
Contributor

@abelaska abelaska commented Dec 7, 2010

Hi,
i have created simple mavenizing of project, so maven users can also easily use Kafka library.

@maguro
Copy link

maguro commented Dec 8, 2010

Why is all the maven stuff in a separate directory? Why is the install sh script needed.

@abelaska
Copy link
Contributor Author

abelaska commented Dec 8, 2010

Maven is in separate directory because i dont want to force anybody to use maven, discussion on what project manager to use is in progress http://groups.google.com/group/kafka-dev/browse_thread/thread/1f2c2f1d5c0d2bea.

And instal script is obsolete now.

@maguro
Copy link

maguro commented Dec 8, 2010

Putting a POM file in the root directory doesn't force anyone to use maven. Alternatively, putting maven in a separate directory makes it seem overly complicated when it is not.

@abelaska
Copy link
Contributor Author

abelaska commented Dec 8, 2010

You're right, i'll put pom.xml into the root directory. No problem.

@maguro
Copy link

maguro commented Dec 9, 2010

What does the change to the zkclient library have to do with Maven?

@abelaska
Copy link
Contributor Author

abelaska commented Dec 9, 2010

It's dependency that is not hosted in central repository, so maven will install this library into the local repository before compilation. It's version with latest fixes. If you insist on version 0.1.0 i have no problem with that.

@maguro
Copy link

maguro commented Dec 9, 2010

Why did you rename the zookeeper jar?

@abelaska
Copy link
Contributor Author

abelaska commented Dec 9, 2010

I have not just renamed it i have used newest version of zookeeper. Is it not obvious?

@maguro
Copy link

maguro commented Dec 9, 2010

No, it looks like all you did was renamed it.

@abelaska
Copy link
Contributor Author

abelaska commented Dec 9, 2010

Well, commit message "Upgrade to ZooKeeper version 3.3.2." is easy to understand i guess.

@maguro
Copy link

maguro commented Dec 9, 2010

Yeah, that's the commit message, not necessarily what happened. However, it seems that git hub erroneously states that the file did not change.

@abelaska
Copy link
Contributor Author

abelaska commented Dec 9, 2010

I understand, trust no one :)

@abelaska
Copy link
Contributor Author

abelaska commented Jan 6, 2011

Any progress in rewieving my patches?

@cburroughs
Copy link
Contributor

@cburroughs
Copy link
Contributor

Hi abelaska, I think at this point sbt's make-pom command produces something reasonable. Do you still think a manually managed pom for compilation is worthwhile? If so could you open a ticket on the new Apache Issue tracker: https://issues.apache.org/jira/browse/KAFKA ?

@abelaska
Copy link
Contributor Author

abelaska commented Aug 3, 2011

Hi,
i guess it's not necessary at that moment. This issue can be closed.

Cheers,
Alois Belaska

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

3 participants