Re: [ANNOUNCE qgit-1.0rc1]

Re: [ANNOUNCE qgit-1.0rc1]

[Pavel Roskin]

Hello, Marco!

On Sun, 2005-12-18 at 16:06 +0100, Marco Costalba wrote:
> A good amount of small fixes and just few non intrusive features
> added, biggest one are speed-up of ref reading at startup, a per
> repository charset encoding using i18n.commitencoding git config
> variable and an annotation progress bar.
> 
> I plan to relase 1.0 for end of next week, so I would like to catch
> last minutes bugs with this rc1.

OK, here you go.  qgit just crashed on me.  I opened the patch pane, the
tree pane and the annotate window.  While the annotate window was
focused, I tried to select multiple files in the tree pane.  Then I
reproduced the crash in the debugger with qgit compiled without
optimization.


Program received signal SIGSEGV, Segmentation fault.
[Switching to Thread 46912496291552 (LWP 19362)]
0x0000000000415943 in Annotate::doAnnotate (this=0x7e6bd0, fileName=@0x7fffffc81f70, 
    sha=@0x7e2dc0) at src/annotate.cpp:102
102                     if (!pa->isValid) {
(gdb) p pa
$1 = (FileAnnotation *) 0x0
(gdb) bt
#0  0x0000000000415943 in Annotate::doAnnotate (this=0x7e6bd0, fileName=@0x7fffffc81f70, 
    sha=@0x7e2dc0) at src/annotate.cpp:102
#1  0x000000000041596d in Annotate::doAnnotate (this=0x7e6bd0, fileName=@0x7fffffc81f70, 
    sha=@0x7e4d10) at src/annotate.cpp:103
#2  0x000000000041596d in Annotate::doAnnotate (this=0x7e6bd0, fileName=@0x7fffffc81f70, 
    sha=@0x7e27d0) at src/annotate.cpp:103
#3  0x000000000041596d in Annotate::doAnnotate (this=0x7e6bd0, fileName=@0x7fffffc81f70, 
    sha=@0x8db560) at src/annotate.cpp:103
#4  0x000000000041596d in Annotate::doAnnotate (this=0x7e6bd0, fileName=@0x7fffffc81f70, 
    sha=@0x872bc0) at src/annotate.cpp:103
#5  0x000000000041596d in Annotate::doAnnotate (this=0x7e6bd0, fileName=@0x7fffffc81f70, 
    sha=@0x7b5c80) at src/annotate.cpp:103
#6  0x000000000041596d in Annotate::doAnnotate (this=0x7e6bd0, fileName=@0x7fffffc81f70, 
    sha=@0x7c2a50) at src/annotate.cpp:103
#7  0x000000000041596d in Annotate::doAnnotate (this=0x7e6bd0, fileName=@0x7fffffc81f70, 
    sha=@0x790c20) at src/annotate.cpp:103
#8  0x000000000041596d in Annotate::doAnnotate (this=0x7e6bd0, fileName=@0x7fffffc81f70, 
    sha=@0x7b9a30) at src/annotate.cpp:103
#9  0x000000000041596d in Annotate::doAnnotate (this=0x7e6bd0, fileName=@0x7fffffc81f70, 
    sha=@0x7e3d58) at src/annotate.cpp:103
#10 0x0000000000415f5f in Annotate::run (this=0x7e6bd0, fileName=@0x7fffffc81f70, 
    histRevOrder=@0x7fffffc81f80, annotateOk=@0x7fffffc81f5f, canceled=@0x7fffffc81f5e)
    at src/annotate.cpp:73
#11 0x0000000000435a54 in Git::runAnnotate (this=0x6cb5b0) at src/git.cpp:166
#12 0x00000000004832ab in Git::qt_invoke (this=0x6cb5b0, _id=9, _o=0x7fffffc82070)
    at src/moc_git.cc:132
#13 0x000000308a452d3a in QObject::activate_signal () from /usr/lib64/qt-3.3/lib/libqt-mt.so.3
#14 0x000000308a760923 in QSignal::signal () from /usr/lib64/qt-3.3/lib/libqt-mt.so.3
#15 0x000000308a46c581 in QSignal::activate () from /usr/lib64/qt-3.3/lib/libqt-mt.so.3
---Type <return> to continue, or q <return> to quit---
#16 0x000000308a473d09 in QSingleShotTimer::event () from /usr/lib64/qt-3.3/lib/libqt-mt.so.3
#17 0x000000308a3f7cfa in QApplication::internalNotify ()
   from /usr/lib64/qt-3.3/lib/libqt-mt.so.3
#18 0x000000308a3f8674 in QApplication::notify () from /usr/lib64/qt-3.3/lib/libqt-mt.so.3
#19 0x000000308a3edc4c in QEventLoop::activateTimers ()
   from /usr/lib64/qt-3.3/lib/libqt-mt.so.3
#20 0x000000308a3a9d3b in QEventLoop::processEvents () from /usr/lib64/qt-3.3/lib/libqt-mt.so.3
#21 0x000000308a40de37 in QEventLoop::enterLoop () from /usr/lib64/qt-3.3/lib/libqt-mt.so.3
#22 0x000000308a40dd42 in QEventLoop::exec () from /usr/lib64/qt-3.3/lib/libqt-mt.so.3
#23 0x000000000046e976 in main (argc=1, argv=0x7fffffc82688) at src/qgit.cpp:29
(gdb)

That's Fedora Core 4 on x86_64.

Also, qgit doesn't look well with large fonts (not excessively large,
mind you, just with Sans 12):

http://red-bean.com/proski/qgit/

I see sizes are hardcoded in the *.ui files.  Probably there should be a
way to make the widgets resize together with the font.

I don't want to be asked about range every time I start qgit.  Can it be
an option?  In half of the cases I just want to see commits in the last
2-4 days.  In other cases, I want to see the whole history.  Tags are
usually irrelevant.

The "Show All" checkbox in the range select dialog doesn't seem to have
any effect.  I think it should disable "top" and "bottom" comboboxes.

Finally, if I close the range select window by using the window manager
button, I don't want qgit to continue.  Maybe it's just because I'm
annoyed by the useless range dialog :-)

-- 
Regards,
Pavel Roskin

Re: [ANNOUNCE qgit-1.0rc1]

[Marco Costalba]

> OK, here you go.  qgit just crashed on me.  I opened the patch pane, the
> tree pane and the annotate window.  While the annotate window was
> focused, I tried to select multiple files in the tree pane. 
 
Hi Pavel,

 thanks for crash report, I will investigate it.
> 
> Also, qgit doesn't look well with large fonts (not excessively large,
> mind you, just with Sans 12):
> 
> http://red-bean.com/proski/qgit/
> 
> I see sizes are hardcoded in the *.ui files.  Probably there should be a
> way to make the widgets resize together with the font.
> 

I will investigate what Qt designer allows abot font resizing.


> I don't want to be asked about range every time I start qgit.  Can it be
> an option?  In half of the cases I just want to see commits in the last
> 2-4 days.  In other cases, I want to see the whole history.  Tags are
> usually irrelevant.
> 

I know what you mean ;-)

To avoid range select dialog simply call qgit with some command line option (arguments are feeded
to git-rev-parse, so the same options of gitk are allowed), qgit shows the dialog only if called
with no arguments.

If you don't want to type options just call qgit and press enter as soon as range select appears, 
by default you will see commits from HEAD to latest tag.

If, instead, you want to see whole the history, press "Canc" to delete "to" field tag and then
press enter. That's the reason the focus is on "to" field by default ;-)

An option at startup to bybass range select could be an idea tough.


> The "Show All" checkbox in the range select dialog doesn't seem to have
> any effect.  I think it should disable "top" and "bottom" comboboxes.
> 

"Show All" is simply a shortcut for git-rev-parse --show-all option, to see all the branches
between selected commit range.

Tags selection is to define selected commit range

Peraphs it should be better call it "Show all branches" and add a new check "Show whole history"


Thanks for your feedback.

Marco



__________________________________________________
Do You Yahoo!?
Tired of spam?  Yahoo! Mail has the best spam protection around 
http://mail.yahoo.com 

[ANNOUNCE qgit-1.0rc1]

[Marco Costalba]

A good amount of small fixes and just few non intrusive features
added, biggest one are speed-up of ref reading at startup, a per
repository charset encoding using i18n.commitencoding git config
variable and an annotation progress bar.

I plan to relase 1.0 for end of next week, so I would like to catch
last minutes bugs with this rc1.

Thanks to Pavel and Junio for fixes and suggestions.


DOWNLOAD

Download tarball from sourceforge as usual:
http://prdownloads.sourceforge.net/qgit/qgit-1.0rc1.tar.bz2?download

Or directly from git repository:
http://digilander.libero.it/mcostalba/qgit.git

With Junio help, I setup the repository to work both with cg-clone and
git-clone and also
tags fetching is available now ;-)


INSTALLATION

You need scons and qt-mt developer libs, version 3.3.4 or better,
already installed.

qgit is NOT compatible with Qt4.

On some platforms (Debian) you should set QTDIR before to compile.

- unpack tar file
- make
- make install

qgit will be installed in $HOME/bin


CHANGELOG

- reselect current revision after a refresh or a tree filter/un-filter
- speed up startup refs reading deferring tag messages fetching
- print offending command line when running commands synchronously
- added fancy annotation progress bar
- use git variable i18n.commitencoding to store charset codec name
- converted to use git-peek-remote to read refs: slower but cleaner
- added python and perl file icons in tree view
- fix a long standing bug that filters out error reporting when a
  process writes to stderr
- fix tree filtering when tree selected item is not visible
- fix error getting file sha
- add wait cursor while opening tree directory
- fix issue with mouse right-click that could freeze the UI in some cases
- always load names cache file if present
- fix graph in (rare) case of initial rev to be a fork
- fix enable/disable 'view diff' and 'view file' menus policy
- fix multi-tag commits display in status bar
- remove PGP signature from tag message
- fix file viewer sometimes opens on a wrong file when file is tree selected


Marco

	

	
		
___________________________________ 
Yahoo! Mail: gratis 1GB per i messaggi e allegati da 10MB 
http://mail.yahoo.it