Discussion:
New plugins in 7.1.x
Pushkar Pradhan
2018-11-28 20:06:49 UTC
Permalink
Hello,
I would like to opensource a plugin we are using into 7.1.x branch.
Please let me know if this would be acceptable?
--
pushkar
Susan Hinrichs
2018-11-28 22:27:51 UTC
Permalink
Do you have a PR for this plugin?
Post by Pushkar Pradhan
Hello,
I would like to opensource a plugin we are using into 7.1.x branch.
Please let me know if this would be acceptable?
--
pushkar
Pushkar Pradhan
2018-11-28 22:30:23 UTC
Permalink
Not yet. I am cleaning up the code for open sourcing.
Post by Susan Hinrichs
Do you have a PR for this plugin?
Post by Pushkar Pradhan
Hello,
I would like to opensource a plugin we are using into 7.1.x branch.
Please let me know if this would be acceptable?
--
pushkar
--
pushkar
Bryan Call
2018-11-29 18:10:35 UTC
Permalink
Is the plugin going to introduce any new dependencies?

-Bryan
Post by Pushkar Pradhan
Hello,
I would like to opensource a plugin we are using into 7.1.x branch.
Please let me know if this would be acceptable?
--
pushkar
Pushkar Pradhan
2018-11-29 18:45:29 UTC
Permalink
The plugin has dependencies on libyaml and libpcre.
AFAIK TS already depends on these two.
Post by Bryan Call
Is the plugin going to introduce any new dependencies?
-Bryan
Post by Pushkar Pradhan
Hello,
I would like to opensource a plugin we are using into 7.1.x branch.
Please let me know if this would be acceptable?
--
pushkar
--
pushkar
Bryan Call
2018-11-29 21:14:50 UTC
Permalink
As the secondary release manager I am OK with adding this to 7.1.x. However, it would be good to hear from the primary release manager (Leif).

-Bryan
Post by Pushkar Pradhan
The plugin has dependencies on libyaml and libpcre.
AFAIK TS already depends on these two.
Post by Bryan Call
Is the plugin going to introduce any new dependencies?
-Bryan
Post by Pushkar Pradhan
Hello,
I would like to opensource a plugin we are using into 7.1.x branch.
Please let me know if this would be acceptable?
--
pushkar
--
pushkar
Leif Hedstrom
2018-11-29 21:15:07 UTC
Permalink
Post by Pushkar Pradhan
The plugin has dependencies on libyaml and libpcre.
AFAIK TS already depends on these two.
Not exactly. libpcre, yes. But the YAML dependency is only in 8.0.x and later.

In general, I’m mostly -1 on adding new features to 7.1.x at this point, so it’ll have to be a very strong argument and completely seamless integration. People should just upgrade to v8.1.0 for new features that didn’t make it into 8.0.0 :-).

Cheers,

— leif
Post by Pushkar Pradhan
Post by Bryan Call
Is the plugin going to introduce any new dependencies?
-Bryan
Post by Pushkar Pradhan
Hello,
I would like to opensource a plugin we are using into 7.1.x branch.
Please let me know if this would be acceptable?
--
pushkar
--
pushkar
Alan Carroll
2018-11-29 21:22:16 UTC
Permalink
I think libyaml and YAMLCPP are different dependencies. Given the goal of
this work, I think it should be fine to make it compatible with 7.1.x
without putting it in the actual distribution, and then look at adding it
to 9.0 with a conversion to YAMLCPP.

Loading...