Bug 192343 - New port: shells/fish-devel - development version of the friendly interactive shell
Summary: New port: shells/fish-devel - development version of the friendly interactive...
Status: Closed Overcome By Events
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: John Marino
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-08-02 19:23 UTC by Sascha Grunert
Modified: 2014-09-09 10:10 UTC (History)
1 user (show)

See Also:


Attachments
the port shar file (26.58 KB, application/x-shar)
2014-08-02 19:23 UTC, Sascha Grunert
no flags Details
updated shar (26.71 KB, application/x-shar)
2014-08-03 14:02 UTC, Sascha Grunert
no flags Details
poudriere testport logs (336.50 KB, application/x-gzip)
2014-08-03 15:32 UTC, Sascha Grunert
no flags Details
the logfile (86.67 KB, text/plain)
2014-08-03 16:23 UTC, Sascha Grunert
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Sascha Grunert 2014-08-02 19:23:15 UTC
This new port references the development version (master on github) of the fish shell.
Comment 1 Sascha Grunert 2014-08-02 19:23:57 UTC
Created attachment 145257 [details]
the port shar file
Comment 2 John Marino freebsd_committer freebsd_triage 2014-08-03 12:16:49 UTC
Can you please provide either)

1) "poudriere testport" or "poudriere bulk -t" logs (preferred)
2) redports or tinderbox logs
3) "make check-plist" followed by "make stage-qa" output

to provide confidence in the quality of the submission
Comment 3 Sascha Grunert 2014-08-03 14:02:49 UTC
Created attachment 145301 [details]
updated shar
Comment 4 Sascha Grunert 2014-08-03 15:32:07 UTC
Created attachment 145305 [details]
poudriere testport logs
Comment 5 Sascha Grunert 2014-08-03 15:32:44 UTC
Attached the poudriere testport log files. Had to remove the gcc build log because this file was too big.
Comment 6 John Marino freebsd_committer freebsd_triage 2014-08-03 15:40:44 UTC
I assume the updated shar came as a result of running these tests?

Weird log, it seems to wrapped in html
Also, it seems that "testport" or "bulk -t" options wasn't used?  I don't see the stage-qa tests.

Anyway, call it close enough.  Moving to patch-ready.
Comment 7 Sascha Grunert 2014-08-03 15:50:00 UTC
Yes, I needed to update the shar file because of the failing QA test.
I run the poudriere testport command. I think file you wanted to see is
2014-08-03_16h13m55s/logs/fish-2.0.20140802.log
inside the tarbarll?
Comment 8 John Marino freebsd_committer freebsd_triage 2014-08-03 15:52:33 UTC
that explains it, I thought it was log.gz, not tar.gz so I just ungzipped it and looked at it.

I'm still wondering why I don't see the stage-qa tests, maybe testport does those checks directly and bulk -t uses bsd.ports.mk to do it.
Comment 9 Sascha Grunert 2014-08-03 15:54:49 UTC
What about fish-2.0.20140802.log line 743?
Comment 10 John Marino freebsd_committer freebsd_triage 2014-08-03 16:10:51 UTC
ugh, now I under the comment about the gcc log.  You archived the entire run.  All was asking for was the fish-devel log (a mere 86k)

Yeah, that's good.  I was obviously looking at a different log as I was only expecting one.

For the benefit of the committer that takes this PR, you might want to attach just the fish log and obsolete the compressed tarball.
Comment 11 John Marino freebsd_committer freebsd_triage 2014-08-03 16:11:25 UTC
(In reply to John Marino from comment #10)
> ugh, now I under the comment about the gcc log. 

now I *understand* the comment ...
Comment 12 Sascha Grunert 2014-08-03 16:23:31 UTC
Created attachment 145307 [details]
the logfile

I will make it clean and attached only the fish logfile. Sorry, but this was my first port.
Comment 13 John Marino freebsd_committer freebsd_triage 2014-08-09 23:34:37 UTC
i never noticed what the port actually was.

Why are you created a -devel version of fish instead of updating the stable version?  What is the -devel version?  (alpha, beta, etc). 

What FreeBSD users are going to use it over stable?

For full disclosure, I am anti-devel ports in general.  In some cases there is a clear need but in most I don't see it and you didn't explain "why" when you opened the PR, only "what".
Comment 14 John Marino freebsd_committer freebsd_triage 2014-08-16 20:47:14 UTC
I'm taking this - I'm not comfortable with it listed as patch-ready without knowing why fish-devel is needed beside fish.  So I'll hold it while waiting for for that answer.
Comment 15 John Marino freebsd_committer freebsd_triage 2014-08-23 15:59:20 UTC
Hi Sascha, I've waited 2 weeks for a response.  Are you around?
Comment 16 Sascha Grunert 2014-08-23 16:20:13 UTC
Hi John,

Now I'm here to answer your questions: The devel version is the cutting edge version from github (pointing to a SHA). I created this version because the last release of the fish shell was very long ago (nearly one year) and they implemented a lots of features which we can't access with the current release (for example vi key bindings).

Best regards,
Sascha
Comment 17 John Marino freebsd_committer freebsd_triage 2014-08-23 16:27:40 UTC
You are not the same person that maintains shells/fish, correct?

I really want to avoid two versions, ideally this could replace shells/fish.  But I need to know if/when a release is planned (e.g. no point if new fish is released in 1 month) and then I have no idea about the stability/quality of a github snapshot.

Have you coordinated with the fish maintainer at all?  There is a school of though that says -devel versions have to be maintained by the same person that maintains the stable version.  Right now this is not a rule, but some people want it to be.  You can imaging 2 versions that are not coordinated are a burden for us.
Comment 18 Sascha Grunert 2014-08-23 16:29:37 UTC
Okay then I try to find out when a new release will come up.
Comment 19 Sascha Grunert 2014-08-27 15:18:59 UTC
Here is the conversation on the fish mailing list:

On Sun, 24 Aug 2014, Sascha Grunert wrote:
> When is an upcoming release planned? The current version is a little bit 
> outdated in comparison the Github progress.

I'm still trying to get 2.1.1 out the door (I'm on vacation at present and 
the intersection of internet connectivity and free time is very small). I 
keep making promises I can't keep about "this weekend", but maybe this 
weekend... I've pushed all the patches to Integration_2.1.1, so it's ready 
for sign-off, but haven't written release notes or prepared the new 
packages yet - it will require a fishd restart so there is a bit more than 
usual involved.

After that I am hoping to look at 2.2.0. There are a few rough edges that 
will need polishing first, but there are certainly lots of new features 
that it would be good to get into people's hands.

Cheers

David Adam
fish committer
Comment 20 Sascha Grunert 2014-08-27 15:19:13 UTC
Here is the conversation on the fish mailing list:

On Sun, 24 Aug 2014, Sascha Grunert wrote:
> When is an upcoming release planned? The current version is a little bit 
> outdated in comparison the Github progress.

I'm still trying to get 2.1.1 out the door (I'm on vacation at present and 
the intersection of internet connectivity and free time is very small). I 
keep making promises I can't keep about "this weekend", but maybe this 
weekend... I've pushed all the patches to Integration_2.1.1, so it's ready 
for sign-off, but haven't written release notes or prepared the new 
packages yet - it will require a fishd restart so there is a bit more than 
usual involved.

After that I am hoping to look at 2.2.0. There are a few rough edges that 
will need polishing first, but there are certainly lots of new features 
that it would be good to get into people's hands.

Cheers

David Adam
fish committer
Comment 21 John Marino freebsd_committer freebsd_triage 2014-08-27 15:42:16 UTC
ok, so my interpretation of this is that we forgo this -devel port but use the work you did as a PR against the stable version.

Right?  Presumably there's only a minimal change for this proposal to handle the next release so most of the work to update the port is already done and only the maintainer of the stable port needs to sign off on it.
Comment 22 Sascha Grunert 2014-08-27 16:21:06 UTC
Alright then we should do it this way.
Comment 23 John Marino freebsd_committer freebsd_triage 2014-09-09 10:10:47 UTC
Okay, the plan is that sascha will submit a PR against the current shells/fish when the next release comes.

Feel free to add me to the CC and I will take care of that PR after the current maintainer concurs with it.