damienhaynes / moving-pictures

Moving Pictures is a movies plug-in for the MediaPortal media center application. The goal of the plug-in is to create a very focused and refined experience that requires minimal user interaction. The plug-in emphasizes usability and ease of use in managing a movie collection consisting of ripped DVDs, and movies reencoded in common video formats supported by MediaPortal.
12 stars 6 forks source link

MPEI v2 (patch included) #1044

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
I managed to create a MPEI of Moving Pictures that does everything the NSIS 
installer does.

MpeMaker works fine with relative paths, and almost everything can be done with 
the GUI.
Stuff like backing up the database has to be done with a script,
but so does NSIS.

Included in the patch is the .xmp2, a logo.png, as header image in MPEI does 
not support .ico :( , and a build batch that builds plugin and MPEI (it assumes 
version.txt is already present to set correct version).

I know we barely have any docs for MPEI which is the primary reason you've been 
against on using it and I hope to improve that soon. In fact, as a starting 
point, while doing this installer, I used the Windows 7 Steps Recorder to 
record every click, every keystroke I made to take a look at and build 
documentation from that. I even uploaded it so anyone can take a look if the 
want to. (Warning: there's over 200 steps/screenshots.)

https://skydrive.live.com/?cid=968ed5b5fac2436c&sc=documents&uc=6&id=968ED5B5FAC
2436C%2148714#cid=968ED5B5FAC2436C&id=968ED5B5FAC2436C%2148714&sc=documents

Meanwhile to answer question from issue #904, most of the issues have been 
resolved for awhile or are resolved for MP 1.2:
-MpeMaker does work with relative paths
-There is a command line switch for MpeMaker. The build batch file in the patch 
uses these.
-There's really no need to edit xml files manually so documenting every detail 
is unimportant I think.
-Mostly everything can be done on GUI and for the things that it can't do then 
scripts are available which are simply C# scripts. So learning curve should be 
low.
-We've improved on the installer UI's front. I know there's still stuff to do 
and for MP 1.3 will work heavily on that. I hope to receive specific feedback 
on our forums for this once 1.2 is released.

PS I know the png does not have transparency but that's because I used a crappy 
tool to convert the image.

Original issue reported on code.google.com by migue333@gmail.com on 21 Aug 2011 at 4:45

Attachments:

GoogleCodeExporter commented 9 years ago
Thanks migue333, your work is much appreciated. I am a bit busy this weekend 
with ludum dare, but I will take a look at this this week as things slow down. 
I really would like to move to a more official installer but MPEI has been 
lacking in the past. Hopefully things are looking a little more polished these 
days.

Original comment by conrad.john on 21 Aug 2011 at 5:01

GoogleCodeExporter commented 9 years ago
Was pointed out a bug in the backup script. I read the "InstallCount" value 
from registry as int instead of a string, and also stored it as an int instead 
of string.

Original comment by migue333@gmail.com on 23 Aug 2011 at 8:29

GoogleCodeExporter commented 9 years ago
also, the icon seems to point to the wrong location: 

2011-09-16 13:51:40.480057 [Warn.][MPMain(1)]: [MPEI] Failed to download update 
file from http://code.google.com/p/moving-pictures/svn/trunk/Setup/logo.ico: 
The remote server returned an error: (404) Not Found.

Original comment by bgmei...@gmail.com on 16 Sep 2011 at 12:04

GoogleCodeExporter commented 9 years ago
Attached is a new patch with a few fixes (registry cast error, wrong icon 
location, added logo.png,...)

I also created a compiled version to try for yourself: 
http://www.2shared.com/file/h8o_ZYPS/moving-pictures-123-setup.html

Original comment by bgmei...@gmail.com on 28 Sep 2011 at 5:47

GoogleCodeExporter commented 9 years ago

Original comment by conrad.john on 30 Dec 2011 at 6:50

GoogleCodeExporter commented 9 years ago
yay!

Original comment by datahaza...@gmail.com on 31 Dec 2011 at 6:35

GoogleCodeExporter commented 9 years ago

Original comment by conrad.john on 20 Oct 2012 at 5:45

GoogleCodeExporter commented 9 years ago

Original comment by conrad.john on 20 Oct 2012 at 5:46

GoogleCodeExporter commented 9 years ago
I did not look at patch attached, but I believe what I have already implemented 
for 1.6 covers this requirement.

Original comment by damien.haynes@gmail.com on 11 Jan 2014 at 10:06