BASH Portable Dev. Test 5

When I try to download from the mediafire page that opens when I select the link above, I'm getting spam pages and advertising and no download. I'd appreciate a confirmation that this is a valid download link.

Works great.

Downloaded!!!

if you want, i can mirror the download, cut out any problems you might be having
PM me on the IRC sometime, even when i'm not in the room, I'm always on the network.

what is the non-portable version of "BASH Portable"? it looks a lot like "GNU utilities for Win32":

http://unxutils.sourceforge.net/

thanks,
david

General Stuff
• OS: Windows XP sp3
• Rights: Admin
• What Release(Dev test # or Pre-Release #): DT3

Paf specific Information

main dir

• HTml problem. read note about it at the bottom
• App dir
  • you need to add the readme.txt
  • AppInfo dir
    • change the icon and remove the 16 color versions
  • Appinfo.ini
    • Change the publisher to daBomb69 & PortableApps.com
    • Change Display version to 1.0.0.0
• BASH dir
  • All Clean
• Default Data dir
  • you have an "anotherdirectory" and a settings what is the purpost of the another directory?
  • In the settings directory you have a file called "ChrisPortableApps.comLauncherTestSettings.ini" I dont think you need that

General Errors

General Errors
1. I think the .bash_history showed up on my C: drive cause i ran it from my c drive, so while it is still technically on the drive, it really should be moved back into the data folder.
2. while I ran BP, i found that it was never really saving the information in the DATA folder.

OH i have the help.html file redone - will post a link to it soon

dabomb, you forget the ul tags.
And, bummer, I was gonna do a package with unixtils Oh well. I think mine is fundamentally different from yours. Guess that means I'll have to download and see.

bash history is not preserved, that is good.

aight just a couple things.
Make sure u add the readme in the app directory (i am not sure if u still need this so if someone could just say so that would b great)
Also i think to maintain paf u need the default data dir in the APP dir, so just re-add that.
Finally add the correct ini file to the Other\Source dir.

good job with this

Regards
~powerjuce

Okay, I anylzed this.

In your source, you need to have your copyright, john's copyright, and the general GNU license header:

;Website: http://PortableApps.com/BASHPortable

;This software is OSI Certified Open Source Software.
;OSI Certified is a certification mark of the Open Source Initiative.

;This program is free software; you can redistribute it and/or
;modify it under the terms of the GNU General Public License
;as published by the Free Software Foundation; either version 2
;of the License, or (at your option) any later version.

;This program is distributed in the hope that it will be useful,
;but WITHOUT ANY WARRANTY; without even the implied warranty of
;MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
;GNU General Public License for more details.

;You should have received a copy of the GNU General Public License
;along with this program; if not, write to the Free Software
;Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.

Legal Copyright should be LegalCopyright "PortableApps.com & Contributors", not John T. Haller.

Section PATH
  ReadEnvStr $R1 PATH
  StrCpy $R1 "$R1;$EXEDIR\App\Bash\usr\local\wbin"
  ${Locate} "$EXEDIR\App\Bash\usr\local\wbin" "/L=D /G=0" "AddDirectoryToPath"
  System::Call 'Kernel32::SetEnvironmentVariableA(t, t) i("PATH", R1).n'
SectionEnd

Function AddDirectoryToPath
  StrCpy $R1 "$R1;$R9"
  Push ""
FunctionEnd

Why not just System::Call 'Kernel32::SetEnvironmentVariableA(t, t) i("PATH", $EXEDIR\App\Bash\usr\local\wbin" ).n'?

You should have the standard messages about things not being found, etc.

You need TheEnd:
newadvsplash::stop /WAIT
so that the splash dll will be deleted and not left behind.

Rename `C:\.bash_history` `$EXEDIR\Data\.bash_history` rename doesn't work across drives.

In the appinfo.ini
Publisher=daBomb69 & PortableApps.com should be Win-Bash &PortableApps.com

[Version]
PackageVersion=1.0.0.0
DisplayVersion=1
Should be the version of WinBash and a short version of Win-Bash, i.e. 1.6.4.0. and 1.6

Your help.html is damaged. The bottom part doesn't show, both IE and Firefox.

I noticed you don't have langauge switching. If WinBash doesn't support it, then that's fine. If it does (and I don't know) you will need to support it.

I've used unxutils for many years but the version of sed included is very much out of date compared the Gnu utils version. I think it should be possible to include the Gnu versions instead. They have more features and are also the versions commonly found in Linux distros.

They can be downloaded from http://gnuwin32.sourceforge.net

disclaimer: I know next to nothing about NSIS, so the remarks below may seem quaintly easy to remedy to those who are so knowledgeable. If so, would someone so apprise of the remedy on this forum. Better--any such remedies should be a part of the help available for any release.

The path environment variable should be set in an easily modifiable '.ini' file in the executable directory. The current path setting is next to useless; almost no one is seriously going to use those very old UnxUtils--no matter how useful they were in their time--if updated mingw and/or gnuwin32 binaries are available with a simple resetting of the path.