Skip to content

Commit

Permalink
lib: remove shortbanner support
Browse files Browse the repository at this point in the history
This was added 2010-11-02 as an experimental feature by Laurent.
But it was never documented, nor hooked up to anything, or usable
without modifying the library.
  • Loading branch information
fingolfin committed Jan 2, 2017
1 parent 2ffe4ac commit 420df25
Showing 1 changed file with 0 additions and 28 deletions.
28 changes: 0 additions & 28 deletions lib/init.g
Original file line number Diff line number Diff line change
Expand Up @@ -508,20 +508,6 @@ BindGlobal( "ShowKernelInformation", function()

sysdate:= GAPInfo.BuildDateTime;

if IsBound(GAPInfo.shortbanner) then
Print("This is GAP ", GAPInfo.Version, " of ", sysdate,
" (", GAPInfo.Architecture);
if "gmpints" in LoadedModules() then
Print("+gmp");
fi;
if IsBound( GAPInfo.UseReadline ) then
Print("+readline");
fi;
Print(")\n");
if GAPInfo.CommandLineOptions.L <> "" then
Print( "Restoring workspace ", GAPInfo.CommandLineOptions.L, "\n");
fi;
else
indent := " ";
if GAPInfo.TermEncoding = "UTF-8" then
btop := "┌───────┐\c"; vert := ""; bbot := "└───────┘\c";
Expand All @@ -546,7 +532,6 @@ BindGlobal( "ShowKernelInformation", function()
if GAPInfo.CommandLineOptions.L <> "" then
Print( " Loaded workspace: ", GAPInfo.CommandLineOptions.L, "\n" );
fi;
fi;
end );

# delay printing the banner, if -L option was passed (LB)
Expand Down Expand Up @@ -878,18 +863,6 @@ BindGlobal( "ShowPackageInformation", function()
Print( suffix );
end;


if IsBound(GAPInfo.shortbanner) then
indent := " ";
if GAPInfo.PackagesLoaded <> rec() then
print_info( "Packages ",
List( RecNames( GAPInfo.PackagesLoaded ),
name -> Concatenation(
GAPInfo.PackagesLoaded.( name )[3], " ",
GAPInfo.PackagesLoaded.( name )[2] ) ),
".\n" );
fi;
else
indent := " ";

# For each loaded component, print name and version number.
Expand Down Expand Up @@ -928,7 +901,6 @@ BindGlobal( "ShowPackageInformation", function()

Print( " Try '??help' for help. See also '?copyright', '?cite' and '?authors'",
"\n" );
fi;
end );
#T show also root paths?

Expand Down

2 comments on commit 420df25

@hulpke
Copy link
Contributor

@hulpke hulpke commented on 420df25 Jan 4, 2017

Choose a reason for hiding this comment

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

I think this makes sense.

@fingolfin
Copy link
Member Author

Choose a reason for hiding this comment

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

Glad to hear! However, this commit is now outdated, I rewrote the history several times. You can find its current incarnation at #1045 -- I'll now stop rewriting the PR (unless I am asked to make changes).

Please sign in to comment.