Things to include into a package update

[Radek Pazdera]

Hi guys!

Honza found a pretty severe issue in the package for Fedora, just before
the presentation at FOSDEM. With this bug, lnst-ctl will fail
practically every time it is executed.

I already posted the patches to fix it and I plan to create a package
update and submit it later this week, so it is ready before devconf.

Anyway, is there anything else we should include to the package update?

Thanks
-Radek

[Jiri Pirko]

Mon, Feb 04, 2013 at 02:54:30PM CET, rpazdera(a)redhat.com wrote:
> Hi guys!
>
> Honza found a pretty severe issue in the package for Fedora, just before
> the presentation at FOSDEM. With this bug, lnst-ctl will fail
> practically every time it is executed.
>
> I already posted the patches to fix it and I plan to create a package
> update and submit it later this week, so it is ready before devconf.
>
> Anyway, is there anything else we should include to the package update? 
Just push the fix now and we'll deal with the rest later

> Thanks
> -Radek
> _______________________________________________
> LNST-developers mailing list
> LNST-developers(a)lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/lnst-developers

[Jan Tluka]

For anyone who'd like to use the current rpm, there's a simple workaround
to avoid the crash.

Download the following file:
http://git.fedorahosted.org/cgit/lnst.git/plain/test_tools/multicast/lnst...

and copy it to "/usr/share/lnst/test_tools/multicast" directory where
you're running the controller.

This should be the way to go or it's possible to use git version as
well.

-Jan

Mon, Feb 04, 2013 at 02:54:30PM CET, rpazdera(a)redhat.com wrote:
> Hi guys!
>
> Honza found a pretty severe issue in the package for Fedora, just before
> the presentation at FOSDEM. With this bug, lnst-ctl will fail
> practically every time it is executed.
>
> I already posted the patches to fix it and I plan to create a package
> update and submit it later this week, so it is ready before devconf.
>
> Anyway, is there anything else we should include to the package update?
>
> Thanks
> -Radek
> _______________________________________________
> LNST-developers mailing list
> LNST-developers(a)lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/lnst-developers

[Jan Tluka]

Mon, Feb 04, 2013 at 02:54:30PM CET, rpazdera(a)redhat.com wrote:
> Hi guys!
>
> Honza found a pretty severe issue in the package for Fedora, just before
> the presentation at FOSDEM. With this bug, lnst-ctl will fail
> practically every time it is executed.
>
> I already posted the patches to fix it and I plan to create a package
> update and submit it later this week, so it is ready before devconf.
>
> Anyway, is there anything else we should include to the package update?
>  
Actually there is :-)

I've hit another crash because of incorrect mode for file reading.
Non-root user won't be able to use the rpm without the fix [1].

Ok, just noticed that Jirka just applied it. So, let's build a stable
rpm!

[1] https://lists.fedorahosted.org/pipermail/lnst-developers/2013-February/00...